This is an automated email from the ASF dual-hosted git repository. eolivelli pushed a commit to branch MCHECKSTYLE-353 in repository https://gitbox.apache.org/repos/asf/maven-checkstyle-plugin.git
commit ad8f4da564393a3429471e46a9d99db70b922ccf Author: Zoran Regvart <zregv...@apache.org> AuthorDate: Mon May 7 15:33:07 2018 +0200 [MCHECKSTYLE-353] - Don't resolve any dependencies Not sure why resolving dependencies is needed for running Checkstyle, it does slow down the build considerably. For Apache Camel this makes a ten fold difference from ~20mins to ~2mins for running `mvn checkstyle:check`. Seems this was introduced with MCHECKSTYLE-163 but subsequently integration test was removed in MCHECKSTYLE-272 (#42). --- .../apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java b/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java index 8c50001..d171791 100644 --- a/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java +++ b/src/main/java/org/apache/maven/plugins/checkstyle/CheckstyleViolationCheckMojo.java @@ -72,7 +72,7 @@ import com.puppycrawl.tools.checkstyle.api.CheckstyleException; * @author <a href="mailto:joa...@erdfelt.net">Joakim Erdfelt</a> * @version $Id$ */ -@Mojo( name = "check", defaultPhase = LifecyclePhase.VERIFY, requiresDependencyResolution = ResolutionScope.TEST, +@Mojo( name = "check", defaultPhase = LifecyclePhase.VERIFY, requiresDependencyResolution = ResolutionScope.NONE, threadSafe = true ) public class CheckstyleViolationCheckMojo extends AbstractMojo