McFoggy commented on a change in pull request #438: URL: https://github.com/apache/maven/pull/438#discussion_r564504423
########## File path: maven-plugin-api/pom.xml ########## @@ -54,6 +54,11 @@ under the License. <groupId>org.codehaus.plexus</groupId> <artifactId>plexus-classworlds</artifactId> </dependency> + <dependency> Review comment: That was just a simple unit test, as far as possible I didn't wanted to interfere/pollute the console outputs. If it is not a common practice to mock then I can add a stub/custom impl (with control on log level) for the same purpose. ---------------------------------------------------------------- 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