Maven 2.x Checkstyle Plugin

filtering by severity

Details

  • Type: New Feature New Feature
  • Status: Open Open
  • Priority: Major Major
  • Resolution: Unresolved
  • Affects Version/s: None
  • Fix Version/s: None
  • Labels:
    None
  • Number of attachments :
    1

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Carlos Sanchez added a comment -

Just to remember that it's already implemented in m1

Show
Carlos Sanchez added a comment - Just to remember that it's already implemented in m1
Hide
Permalink
Brett Porter added a comment -

yep, I filed it in a review of the m1 plugin

Show
Brett Porter added a comment - yep, I filed it in a review of the m1 plugin
Brett Porter made changes -
Field Original Value New Value
Fix Version/s 2.0-beta-4 [ 11991 ]
Edwin Punzalan made changes -
Assignee Edwin Punzalan [ elpunzalan ]
Edwin Punzalan made changes -
Status Open [ 1 ] In Progress [ 3 ]
Hide
Permalink
Edwin Punzalan added a comment -

Attached a partial patch. The patch enhanced the report by supplying the missing sections from m1 version.

The refactoring will enable filtering by severity. However, I'm still undecided as to how to generate the severity reports.

I am looking at creating separate mojos for each severity report.

Is there a possible way to create a separate html page without creating a separate mojo?

Any advice would be really helpful.

Thanks.

Show
Edwin Punzalan added a comment - Attached a partial patch. The patch enhanced the report by supplying the missing sections from m1 version. The refactoring will enable filtering by severity. However, I'm still undecided as to how to generate the severity reports. I am looking at creating separate mojos for each severity report. Is there a possible way to create a separate html page without creating a separate mojo? Any advice would be really helpful. Thanks.
Edwin Punzalan made changes -
Attachment MNG-1003-maven-checkstyle-plugin.patch [ 16964 ]
Edwin Punzalan made changes -
Status In Progress [ 3 ] Open [ 1 ]
Hide
Permalink
Brett Porter added a comment -

applied

Show
Brett Porter added a comment - applied
Hide
Permalink
John Casey added a comment -

What's the status on this? Are we still waiting to figure out 1:n mappings of mojo:pages? If so, can anyone offer any advice? I'm not too educated on the reporting side of the house, but I'm willing to learn if there are no takers.

Show
John Casey added a comment - What's the status on this? Are we still waiting to figure out 1:n mappings of mojo:pages? If so, can anyone offer any advice? I'm not too educated on the reporting side of the house, but I'm willing to learn if there are no takers.
Hide
Permalink
Edwin Punzalan added a comment -

John,

Yes. There is already a MultipageReport but I don't think its working. That would be a good place to start.

Show
Edwin Punzalan added a comment - John, Yes. There is already a MultipageReport but I don't think its working. That would be a good place to start.
John Casey made changes -
Fix Version/s 2.0.2 [ 12107 ]
Fix Version/s 2.0.1 [ 12000 ]
Jason van Zyl made changes -
Project Maven 2 [ 10500 ] Maven 2.x Checkstyle Plugin [ 11127 ]
Key MNG-1003 MCHECKSTYLE-19
Component/s maven-checkstyle-plugin [ 11791 ]
Workflow Maven [ 39812 ] jira [ 43974 ]
Type Improvement [ 4 ] Bug [ 1 ]
Fix Version/s 2.0.3 [ 12107 ]
Jason van Zyl made changes -
Workflow jira [ 43974 ] Maven [ 44672 ]
Brett Porter made changes -
Workflow Maven [ 44672 ] Maven New [ 51447 ]
Hide
Permalink
Carlos Sanchez added a comment -

I think now the multipage works, so this should be easy to apply

Show
Carlos Sanchez added a comment - I think now the multipage works, so this should be easy to apply
Dennis Lundberg made changes -
Issue Type Bug [ 1 ] New Feature [ 2 ]

People

  • Assignee:
    Edwin Punzalan
    Reporter:
    Brett Porter
Vote (0)
Watch (0)

Dates

  • Created:
    Updated: