psiroky commented on code in PR #205:
URL: https://github.com/apache/maven-enforcer/pull/205#discussion_r1057922768
##########
maven-enforcer-plugin/src/main/java/org/apache/maven/plugins/enforcer/EnforceMojo.java:
##########
@@ -115,6 +120,14 @@ public class EnforceMojo extends AbstractMojo {
@Parameter(required = false, property = "rules")
private String[] commandLineRules;
Review Comment:
The property name `rules` seems wrong. I would expect the `enforcer.` prefix
here.
We can't really rename the property at this point since that would be a
breaking change, but maybe we could deprecate it and introduce new one named
`enforcer.rules` which would be the replacement?
--
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
