Skip to content

Commit c4a93f9

Browse files
author
Guillaume Boué
committed
[MCHECKSTYLE-338] Add support for 'omitIgnoredModules'
Make IT not rely on the 'check' goal invoking the 'checkstyle' goal first. git-svn-id: https://svn.apache.org/repos/asf/maven/plugins/trunk@1798383 13f79535-47bb-0310-9956-ffa450edef68
1 parent 898a9db commit c4a93f9

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/it/MCHECKSTYLE-338/pom.xml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,9 @@
4646
</dependencies>
4747
<executions>
4848
<execution>
49+
<phase>verify</phase>
4950
<goals>
51+
<goal>checkstyle</goal>
5052
<goal>check</goal>
5153
</goals>
5254
</execution>

src/it/MCHECKSTYLE-338/verify.groovy

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ buildLog = new File( basedir, 'build.log' )
2424
assert buildLog.text.contains( 'EmptyLogging: EmptyLoggingCheck on file TestFalse.java' )
2525
assert !buildLog.text.contains( 'EmptyLogging: EmptyLoggingCheck on file TestTrue.java' )
2626

27-
// verify that the "checkstyle" goal launched by "check" does the same
27+
// verify that the "checkstyle" goal does the same
2828
site = new File( basedir, 'omitignoredmodules-false/target/site/checkstyle.html' )
2929
assert site.text.contains( 'EmptyLoggingCheck on file TestFalse.java' )
3030
site = new File( basedir, 'omitignoredmodules-true/target/site/checkstyle.html' )

0 commit comments

Comments
 (0)