Uploaded image for project: 'Maven Checkstyle Plugin'
  1. Maven Checkstyle Plugin
  2. MCHECKSTYLE-338

Add support for 'omitIgnoredModules'

Attach filesAttach ScreenshotVotersWatch issueWatchersCreate sub-taskLinkCloneUpdate Comment AuthorReplace String in CommentUpdate Comment VisibilityDelete Comments
    XMLWordPrintableJSON

Details

    • Improvement
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.17
    • 3.0.0
    • checkstyle:check
    • None
    • Windows 7, 64 bit

    Description

      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.

      Attachments

        Activity
          This comment will be Viewable by All Users Viewable by All Users
          Cancel

          People

            gboue Guillaume Boué
            rveach richard
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Slack

                Issue deployment