[ http://jira.codehaus.org/browse/MCHECKSTYLE-45?page=comments#action_75916 ] Jacob Robertson commented on MCHECKSTYLE-45: --------------------------------------------
Let me describe my situation. First off, we use one common checkstyle.xml definition hosted on an internal http server. This same checkstyle.xml is consumed by both the eclipse-cs plugin (as we develop in eclipse) and by maven both when continuum kicks it off and when we run it locally. However, we were very unhappy with using "error" on any of our checks, because in the IDE it was showing up so as to mask whether it was a true build error or just a checkstyle error. We fought with that for a while, and in the end just changed our checkstyle.xml to declare globally that all checks were just warnings. But now we would also like to explore the possibility of having maven "break the build" when the checkstyle *warnings* show up. Here is the rationale: while developing in eclipse, we want things to show up as warnings to make our IDE experience livable. However, once we go to check something in, we know that all those warnings must be cleared up or we'll break the build. This way, we get the best of both worlds. All I'm trying to do is demonstrate a practical, real-world scenario under which this feature would be useful. > It should be possible to configure checkstyle:check to fail on "warnings". > -------------------------------------------------------------------------- > > Key: MCHECKSTYLE-45 > URL: http://jira.codehaus.org/browse/MCHECKSTYLE-45 > Project: Maven 2.x Checkstyle Plugin > Issue Type: New Feature > Affects Versions: 2.1 > Reporter: Fabian Bauschulte > Attachments: MCHECKSTYLE-45-maven-checkstyle-plugin.patch > > > As mentioned in MCHECKSTYLE-38 it should be possible to configure that > "checkstyle:check" fails on warnings or not. -- 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
