slawekjaranowski commented on code in PR #126:
URL: https://github.com/apache/maven-wrapper/pull/126#discussion_r1567788687
##########
maven-wrapper-plugin/pom.xml:
##########
@@ -34,7 +34,7 @@ under the License.
<description>The Maven Wrapper Plugin is a plugin that provides support for
the Maven Wrapper by unpacking Maven Wrapper Distribution to the current
project.</description>
<prerequisites>
- <maven>${mavenVersion}</maven>
+ <maven>3.2.5</maven>
Review Comment:
I'm still not sure about it ... when we depend on newer version we can have
some issues in runtime which is not cover by ITs.
But from other side we have more information about depractions ...
The perfect way will be for me depends on the same version as in in
prerequisites ... but we block using plugin for older Maven ....
So we need a compromise on it 😄
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
