-
Notifications
You must be signed in to change notification settings - Fork 36
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
Update 2.7 branch to use Quarkus 2.7.6 #686
Conversation
Add check of passing HTTP status error through
#657 was cherry picked in order to fix errors in |
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.
- Quarkus-cli:
Works this test on 2.7.6.Final? https://github.com/quarkus-qe/quarkus-test-suite/blob/main/quarkus-cli/src/test/java/io/quarkus/ts/quarkus/cli/QuarkusCliExtensionsIT.java#L99
If does, could you remove 2.7.5.Final
Disable annotation
- Life application module:
Please also update this reference
https://github.com/quarkus-qe/quarkus-test-suite/blob/main/lifecycle-application/pom.xml#L42
That was already updated, please see changed files. |
I added |
quarkus-cli/src/test/java/io/quarkus/ts/quarkus/cli/QuarkusCliExtensionsIT.java
Show resolved
Hide resolved
quarkus-cli/src/test/java/io/quarkus/ts/quarkus/cli/QuarkusCliExtensionsIT.java
Outdated
Show resolved
Hide resolved
1d0698f
to
8a00c7f
Compare
Quarkus stream 2.7 is not part of code.quarkus.io anymore, thus tests are failing.
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.
LGTM
Summary
Quarkus 2.7.6.Final has been released and branch 2.7 should use it.
Please select the relevant options.
Checklist: