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

checkstyle-aggregate with multi source directories doesn't work

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

Details

    • Bug
    • Status: Open
    • Major
    • Resolution: Unresolved
    • 3.1.0
    • None
    • None
    • Centos 7.4, Maven 3.6.1
      Java : openjdk version "1.8.0_212"

    Description

      Dear Sir or Madam,

       

      While using the feature of checkstyle-aggregate to generate the report with two sub module maven, the report is only displaying the Java files of the "src/main/java" sub folder in the Project Documentation/Project Reports/Checkstyle on the aggregate report.

      The report for the individual sub module in Project Documentation/Project Reports/Checkstyle is diplaying both Java file from the "src/main/java" and the "gen/main/java".

      It would be expected that the aggregate report would contain all the Java files described in each of the sub modules.

      I added the new source folder (gen/main/java) in children poms with the plugin "build-helper-maven-plugin".

      I also tried different configuration with the sourceDirectories tag without success.

      I think my problem is that the aggregate report is generated with the parent module before the build-helper-maven-plugin phase to add source folders in children pom.

      I attached an example that is highlighting the undesired behavior. The cli is "mvn -Pcheckstyle clean compile site"

      It seems to me that the observed behavior is not aligned with the expected functionality of the plugin. I could not find a work around or an alternative to obtain the desired aggregate functionality.

       

      Kind Regards,

       

      Attachments

        Activity

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

          People

            Unassigned Unassigned
            didierc Crest

            Dates

              Created:
              Updated:

              Slack

                Issue deployment