[ 
https://issues.apache.org/jira/browse/MCHECKSTYLE-338?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16046038#comment-16046038
 ] 

Hudson commented on MCHECKSTYLE-338:
------------------------------------

SUCCESS: Integrated in Jenkins build maven-plugins #8995 (See 
[https://builds.apache.org/job/maven-plugins/8995/])
[MCHECKSTYLE-338] Add support for 'omitIgnoredModules'

Make IT not rely on the 'check' goal invoking the 'checkstyle' goal first. 
(gboue: [http://svn.apache.org/viewvc/?view=rev&rev=1798383])
* (edit) maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/pom.xml
* (edit) maven-checkstyle-plugin/src/it/MCHECKSTYLE-338/verify.groovy


> Add support for 'omitIgnoredModules'
> ------------------------------------
>
>                 Key: MCHECKSTYLE-338
>                 URL: https://issues.apache.org/jira/browse/MCHECKSTYLE-338
>             Project: Maven Checkstyle Plugin
>          Issue Type: Improvement
>          Components: checkstyle:check
>    Affects Versions: 2.17
>         Environment: Windows 7, 64 bit
>            Reporter: richard
>            Assignee: Guillaume Boué
>             Fix For: 3.0.0
>
>
> Checkstyle has an option in it's ant process that allows users to avoid 
> executing modules that have a severity set to ignore. It is called 
> `omitIgnoredModules`. It has been in checkstyle already for atleast 2 years.
> See:
> https://github.com/checkstyle/checkstyle/blob/2f3f4b76f04f0b5071abcaed54648a49a0db3160/src/main/java/com/puppycrawl/tools/checkstyle/ant/CheckstyleAntTask.java#L118
> https://github.com/checkstyle/checkstyle/blob/2f3f4b76f04f0b5071abcaed54648a49a0db3160/src/main/java/com/puppycrawl/tools/checkstyle/ant/CheckstyleAntTask.java#L424-L428
> We are in the process of adding this option to our CLI 
> (https://github.com/checkstyle/checkstyle/issues/3566) and would like to see 
> maven-checkstyle-plugin support this feature so users can customize it.
> A simple tag for this option like: 
> `<omitIgnoredModules>false</omitIgnoredModules>` would be enough.
> Thanks.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to