[jira] Commented: (MCHECKSTYLE-45) It should be possible to configure checkstyle:check to fail on warnings.
[ http://jira.codehaus.org/browse/MCHECKSTYLE-45?page=comments#action_79023 ] Mark Roder commented on MCHECKSTYLE-45: --- I have a similar problem/need to Fabian and Jacob. The supplied patch will support what I need to do for my projects and allow different levels of checkstyle alerts to fail the build. 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
[jira] Commented: (MCHECKSTYLE-45) It should be possible to configure checkstyle:check to fail on warnings.
[ 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
[jira] Commented: (MCHECKSTYLE-45) It should be possible to configure checkstyle:check to fail on warnings.
[ http://jira.codehaus.org/browse/MCHECKSTYLE-45?page=comments#action_67183 ] Carlos Sanchez commented on MCHECKSTYLE-45: --- I don't think this should be changed. A warning is by definition just that, a warning. 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 Type: New Feature 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