belingueres commented on a change in pull request #285:
URL: https://github.com/apache/maven/pull/285#discussion_r828724258
##########
File path:
maven-core/src/main/java/org/apache/maven/plugin/internal/ValidatingConfigurationListener.java
##########
@@ -39,14 +43,17 @@
private final Object mojo;
- private final ConfigurationListener delegate;
+ private final MojoDescriptor mojoDescriptor;
+
+ private final Logger logger;
Review comment:
Creating a Composite of ConfigurationListeners certainly make the design
more open to add additional functionality to execute on plugin parameters, but
do you actually are aware of an specific use case on plugin parameters that can
be added here? ValidatingConfigurationListener already cover both deprecations
and print debug info.
--
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
