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
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.
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.
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

People

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

Dates

  • Created:
    Updated: