Skip to content

Conversation

@holly-cummins
Copy link
Contributor

@holly-cummins holly-cummins commented Sep 9, 2025

This won't have a successful preview on the PR request, since it uses main's script, but hopefully after merging previews will start working again.

Sadly, I can't test I've not broken things, for the same reason. :)

See also #2394

Copy link
Collaborator

@gastaldi gastaldi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link

github-actions bot commented Sep 9, 2025

😭 Deploy PR Preview failed.

@holly-cummins holly-cummins merged commit 53f995c into quarkusio:main Sep 9, 2025
1 check passed
@holly-cummins holly-cummins deleted the do-not-preview-everything branch September 9, 2025 14:56
@holly-cummins
Copy link
Contributor Author

Darn. I wonder if I got the script wrong, because the next preview after this still failed with 'application too large,' and I should have shrunk things by at least 50mb. Either that or surge have reduced the limit from 400mb.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants