Skip to content

Conversation

@gbourant
Copy link

No description provided.

@ia3andy
Copy link
Collaborator

ia3andy commented Apr 3, 2025

@gbourant I wonder why we are changing the value here. I think that was meant for the key only and it is a mistake that we changed the value. @edewit?

@gbourant
Copy link
Author

gbourant commented Apr 3, 2025

At the moment, we do allow keys in upper case, see LAUNCH_MODE in quarkus-web-bundler

@ia3andy ia3andy merged commit 88a73f2 into mvnpm:main Jun 23, 2025
3 checks passed
@ia3andy
Copy link
Collaborator

ia3andy commented Jun 23, 2025

@gbourant I am trying to release all this, but release CI is broken...

Today:
https://www.youtube.com/watch?v=AbSehcT19u0&t=4s

@ia3andy
Copy link
Collaborator

ia3andy commented Jun 24, 2025

Ok it's released, now the web-bundler :)

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