psiroky commented on code in PR #213:
URL: https://github.com/apache/maven-enforcer/pull/213#discussion_r1059372306


##########
enforcer-rules/src/main/java/org/apache/maven/plugins/enforcer/RequireJavaVersion.java:
##########
@@ -76,6 +76,7 @@ public void execute(EnforcerRuleHelper helper) throws 
EnforcerRuleException {
                 + " Build: " + detectedJdkVersion.getBuildNumber() + " 
Qualifier: "
                 + detectedJdkVersion.getQualifier());
 
+        super.setMessage(prepareMessage(detectedJdkVersion, getVersion()));

Review Comment:
   Hmm. So would it make sense to override the message only if it has been not 
provided by the rule configuration? Otherwise I am not sure how else would be 
get the additional info in.



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