[ http://jira.codehaus.org/browse/MCHECKSTYLE-51?page=all ]
Olivier Vierlinck updated MCHECKSTYLE-51:
-----------------------------------------
Attachment: MNG-39787-maven-plugin-checkstyle.patch
Here is a proposed implementation, based on two new configuration params:
showNonViolatedRules : @parameter
expression="${checkstyle.sort.rule.summary}" default-value="false"
enableSeveritySummary : @parameter
expression="${checkstyle.enable.severity.summary}" default-value="true"
Default values are set so that the normal behaviour is obtained by default
Note that although provided in one patch, these two 'features' are independent
and could be used/merged/configured independently (the first one, hidding no
violated rules, is very small and impact only a few lines of code (+ supporting
its configuration)
This is my first contribution to this project, let me now if you need something
more: info, doc,...
> Rules summary flexibility:add possibility to hide non-violated rules and to
> sort by descending number of occurences
> -------------------------------------------------------------------------------------------------------------------
>
> Key: MCHECKSTYLE-51
> URL: http://jira.codehaus.org/browse/MCHECKSTYLE-51
> Project: Maven 2.x Checkstyle Plugin
> Issue Type: Improvement
> Affects Versions: 2.1
> Environment: NA
> Reporter: Olivier Vierlinck
> Priority: Minor
> Attachments: MNG-39787-maven-plugin-checkstyle.patch
>
>
> While using maven + Checkstyle on a new project, the amount of
> errors/warnings was quite high, due both because checkstyle was not yet
> configured to match our preferences.. and also to real style errors.
> To refine this, it very useful to tackle first the big problems=the one
> occuring often
> Similarly, it is helpful NOT to see the rules that are not violated, so that
> we can concentrate on the existing problems
--
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators:
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira
