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

Guillaume Boué edited comment on MCHECKSTYLE-338 at 6/11/17 4:56 PM:
---------------------------------------------------------------------

[~michael-o] I've edited the IT in 
[r1798383|http://svn.apache.org/viewvc?rev=1798383&view=rev]. It now passes 
without the current fix for MCHECKSTYLE-314. (I've noticed reverting this 
commit also breaks the one for MCHECKSTYLE-214, which was edited in r1743552 
and r1743822).


was (Author: gboue):
[~michael-o] I've edited the IT in 
[r1798383|http://svn.apache.org/viewvc?rev=1798383&view=rev]. It now passes 
without the current fix for MCHECKSTYLE-314. (I've noticed it also breaks the 
one for MCHECKSTYLE-214, which was edited in r1743552 and r1743822).

> 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