-
Notifications
You must be signed in to change notification settings - Fork 3k
Bump Gradle version to 9.0.0 #49256
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
Bump Gradle version to 9.0.0 #49256
Conversation
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.
@aloubyansky CI fails now with Gradle 9.0 after merging #49224 (I think)
|
It is caused by using removed APIs in Gradle 9:
|
This comment has been minimized.
This comment has been minimized.
cc @aloubyansky |
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.
I don't think we need all the tests but it would be nice to have a couple of sanity checks that we don't break Gradle 8.x for users in the coming months.
I don't think I know how to write such tests. Is there an example? Or maybe leave to the experts @aloubyansky :) BTW, as an example of breaking changes in Gradle, Quarkus 24.0 had a breaking change which uses new APIs introduced in Gradle 8.11. Some users report that they use Gradle 7.6.4 and it is no longer work on Quarkus 24.0: #48829 However, I don't think we introduce any breaking changes with the latest changes and Quarkus should work with Gradle 8.11+. |
I think the idea would be to take a few simple tests from That being said, we need to decide which version we want as the default for 3.26/3.27. |
Once we find some experts, we'll definitely ask for their opinion!
That's a good point. AFAIR, that user was OK upgrading to Gradle 8? So, unless we fix it, we have Gradle 8 and 9 to test. |
… tests with Gradle 8.14
50a072c
to
1a34025
Compare
@Eng-Fouad i rebased your branch and pushed another commit on top to enable a test run with Gradle 8.14 for a selected test. We may need to select some more. |
Status for workflow
|
LGTM. Thanks. |
@Eng-Fouad thanks for all your efforts on this one! |
Uh oh!
There was an error while loading. Please reload this page.