Dufgui commented on a change in pull request #470: URL: https://github.com/apache/maven/pull/470#discussion_r620057883
########## File path: maven-core/src/main/java/org/apache/maven/plugin/MavenPluginValidator.java ########## @@ -19,63 +19,16 @@ * under the License. */ -import java.util.ArrayList; import java.util.List; - import org.apache.maven.artifact.Artifact; import org.apache.maven.plugin.descriptor.PluginDescriptor; /** * MavenPluginValidator */ -public class MavenPluginValidator +public interface MavenPluginValidator { - private final Artifact pluginArtifact; - - private List<String> errors = new ArrayList<>(); - - private boolean firstDescriptor = true; - - public MavenPluginValidator( Artifact pluginArtifact ) - { - this.pluginArtifact = pluginArtifact; - } - - public void validate( PluginDescriptor pluginDescriptor ) - { - /* - * NOTE: For plugins that depend on other plugin artifacts the plugin realm contains more than one plugin - * descriptor. However, only the first descriptor is of interest. - */ - if ( !firstDescriptor ) Review comment: This code seems to be unused because before the MavenPluginValidator is created by each pluginDescriptor in DefaultMavenPluginManager: https://github.com/apache/maven/blob/master/maven-core/src/main/java/org/apache/maven/plugin/internal/DefaultMavenPluginManager.java#L246 I don't think we need it. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org