Le mardi 14 octobre 2014 09:28:21 Olivier Lamy a écrit : > On 14 October 2014 05:01, Hervé BOUTEMY <[email protected]> wrote: > > 2. if we configured Checkstyle to report an error, this means check should > > fail: if you find that it should not fail, please help improve Checkstyle > > configuration by setting severity to warning only > > /me not a checkstyle expert configuration :-) this makes me remember there is a feature improvement for m-checkstyle-p 2.13 that I should explain: I was not a Checkstyle configuration master and felt like you Than I consistently reported Checkstyle rule name and category in every message and report, and a link to Checkstyle documentation for each rule [1]
this gives a good intro to Checkstyle configuration and help a lot when needing to @SuppressWarnings( "checkstyle:name of rule, in lowercase") > /me asking himself if having such hard checkstyle requirement help to > improve user experience. from my perspective, previous feature really improved m-checkstyle-p experience and I know I should submit patches to Checkstyle itself, because I think this could help their users too when using bare-Checkstyle Regards, Hervé [1] http://maven.apache.org/plugins/maven-checkstyle-plugin/checkstyle.html --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
