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

Michael Osipov commented on MCHECKSTYLE-338:
--------------------------------------------

[~gboue], I am about to revert MCHECKSTYLE-314 because it introduces a 
regression. Unfortunately, your IT relies on the broken behavior and will fail. 
Can you locally undo r1720969 and see how you could solve this issue without my 
fix?

> 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