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
Use Maven 3.6.3 in new projects #7253
Conversation
I think there are 3 instances of |
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.
We should also upgrade https://github.com/quarkusio/quarkus/blob/master/bom/runtime/pom.xml#L70 to 3.6.3 or something. We don't really have to be using the minimal supported version ourselves.
@aloubyansky I agree. I'll try to provide another PR bumping that @geoand I believe I replaced all the cases, the other instances seems to be generated from these properties I think |
Indeed the 3rd one does seem to be in a generated file |
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.
Question: is the Maven wrapper used by Quarkus itself 3.6.3? I think we should be consistent as we run the tests with it.
So I would also upgrade the wrapper if not already 3.6.3. Feel free to dismiss this review if it already is.
@gsmet good catch. I updated the wrapper to comply with Maven 3.6.3 too. |
Ah ah, so I killed two birds with one stone because I was thinking of the wrapper we have at the root of the Quarkus project :). |
@gastaldi if you upgrade the maven-core properties, please also upgrade the wrapper used for Quarkus itself here: https://github.com/quarkusio/quarkus/blob/master/.mvn/wrapper/maven-wrapper.properties . |
No description provided.