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


##########
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, but _all plugin builds_ use at least two Maven version: 3.9.6 and ... 
usually 3.6.3. So what "issue in runtime" you expect? Okay, if test coverage is 
poor... but then we fix and improve coverage, and life goes on...



-- 
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]

Reply via email to