Maven Checkstyle Plugin
  1. Maven Checkstyle Plugin
  2. MCHECKSTYLE-172

Checkstyle Plugin 2.8+ generates an additional aggregate report

    Details

    • Type: Bug Bug
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.8, 2.9
    • Fix Version/s: 2.10
    • Labels:
      None
    • Environment:
    • Number of attachments :
      0

      Description

      Using a very simple single module project, the aggregated report is created by default.

      Both the checkstyle and checkstyle-aggregate report are generated.

        Issue Links

          Activity

          Hide
          Justin Ye added a comment -

          Our company wants to upgrade to Checkstyle5.6 for a new feature(accessing suppression file on network).
          Since we are using maven in Jenkins for continious integration, we are looking forward to a coresponding upgrade in maven-checkstyle-plugin so that we can apply the upgrade to the whole company.
          This issue is not critical from my perspective since we have a workaround.
          So is it possible we can move this issue's fix to next release so that 2.10 would not be blocked.
          Many thanks!

          Show
          Justin Ye added a comment - Our company wants to upgrade to Checkstyle5.6 for a new feature(accessing suppression file on network). Since we are using maven in Jenkins for continious integration, we are looking forward to a coresponding upgrade in maven-checkstyle-plugin so that we can apply the upgrade to the whole company. This issue is not critical from my perspective since we have a workaround. So is it possible we can move this issue's fix to next release so that 2.10 would not be blocked. Many thanks!
          Hide
          Dennis Lundberg added a comment -

          Fixed in r1438951.
          I've taken care of the really simple single module case in this fix. Please give it a test.

          If you have more advanced projects that generate an aggregate report when they shouldn't - please open a separate issue for that.

          Show
          Dennis Lundberg added a comment - Fixed in r1438951 . I've taken care of the really simple single module case in this fix. Please give it a test. If you have more advanced projects that generate an aggregate report when they shouldn't - please open a separate issue for that.
          Hide
          Justin Ye added a comment -

          Thank you, Dennis!
          BTW, when will 2.10 be release?

          Show
          Justin Ye added a comment - Thank you, Dennis! BTW, when will 2.10 be release?
          Hide
          Dennis Lundberg added a comment -

          We need to release the plugins-parent first.
          After that I'll release the maven-checkstyle-plugin 2.10.

          Show
          Dennis Lundberg added a comment - We need to release the plugins-parent first. After that I'll release the maven-checkstyle-plugin 2.10.
          Hide
          Justin Ye added a comment -

          So is it possible to have the new version this week? I just want to have a rough date to help me plan the upgrade. Thanks.

          Show
          Justin Ye added a comment - So is it possible to have the new version this week? I just want to have a rough date to help me plan the upgrade. Thanks.

            People

            • Assignee:
              Dennis Lundberg
              Reporter:
              SebbASF
            • Votes:
              4 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: