-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Upgrade Gradle wrapper to 7.0 or greater #19256
Comments
/cc @glefloch, @quarkusio/devtools |
@geoand WDYT ? |
@gsmet were you able to check if this can be done ? (IIRC there could have been issues with the support) |
I'd say we could prepare a PR and we will merge it either for 2.2 or for 2.3. |
Ok I can take care of it next week then. |
Cool, thanks! |
@glefloch we have the green light to move to Gradle 7 in 2.2 if it helps making progress on the JDK 17 support. |
Great I will add this in the branch that enable gradle tests on jdk17 once I’m back from PTO. |
Description
Currently, Gradle 6.9 has some limitations as not being able to define exclusiveContent
This is especially useful to avoid "A confusing dependency" than then later could finish in some security vulnerabilities.
Implementation ideas
Upgrade gradle distribution to something greater or equal to 7
https://github.com/quarkusio/quarkus/blob/main/devtools/gradle/gradle/wrapper/gradle-wrapper.properties#L3
The text was updated successfully, but these errors were encountered: