Skip to content

Commit

Permalink
[MCHECKSTYLE-353] - Don't resolve any dependencies
Browse files Browse the repository at this point in the history
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).
  • Loading branch information
zregvart authored and eolivelli committed May 1, 2019
1 parent f39761f commit 6abad1c
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@
* @author <a href="mailto:joakim@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
Expand Down

0 comments on commit 6abad1c

Please sign in to comment.