cstamas commented on code in PR #126:
URL: https://github.com/apache/maven-wrapper/pull/126#discussion_r1567780642


##########
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:
   Yeah, I did not want to change prerequisite, it was more to kill this 
antipattern to "compile against eons old" that prevents noticing what is being 
deprecated lately in APIs and Core... (and replaced eon old aether w/ 
maven-resolver)



-- 
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: issues-unsubscr...@maven.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to