maven-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael Osipov (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (MCHECKSTYLE-338) Add support for 'omitIgnoredModules'
Date Sun, 11 Jun 2017 10:44:19 GMT

    [ 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)

Mime
View raw message