-
Notifications
You must be signed in to change notification settings - Fork 6
minor: Update to Quarkus 3.20 #441
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR bumps the Quarkus platform version from 3.15.1 to 3.20.0 to stay up-to-date with the latest framework improvements.
- Updated Quarkus platform version property
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for updating Quarkus, @eguicf. Your PR looks good to me. I have two questions but don't think any changes are needed.
Can we wait with merging this PR into main
until the new extension release was published? I'd like to spend some more time using the new Quarkus release in the extension pre-releases before we make it available to users. We could then include it in the next extension release, 1.5.0. WDYT?
src/test/java/io/confluent/idesidecar/websocket/resources/ConnectionEventsTest.java
Outdated
Show resolved
Hide resolved
thank you @flippingbits ! sure, that make sense. |
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the update, @eguicf! I've already approved the PR. Let's wait with merging it until the extension release v1.4.0 is out.
Summary of Changes
addresses #399
Any additional details or context that should be provided?
Pull request checklist
Please check if your PR fulfills the following (if applicable):