[ http://jira.codehaus.org/browse/MCHECKSTYLE-51?page=comments#action_77927 ] Vincent Siveton commented on MCHECKSTYLE-51: --------------------------------------------
Olivier, Thanks! Your patch sounds good (for the next time, please ensure you use our checkstyle [1]). Could you provide us: * small documentation (ie one example) [2] * unit tests with plugin harness [3] Thanks again. [1] http://maven.apache.org/guides/development/guide-m2-development.html#Maven%20Code%20Style [2] http://docs.codehaus.org/display/MAVEN/Maven+Plugin+Documentation [3] http://docs.codehaus.org/display/MAVEN/Maven+Plugin+Harness > 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
