[ http://jira.codehaus.org/browse/MCHECKSTYLE-38?page=all ]
Brett Porter updated MCHECKSTYLE-38:
------------------------------------
Fix Version: 2.1
ok, that certainly could be considered a bug in its current state, and its not
going to regress on someone (the worst is it will fail now and not fail later,
but they are unlikely to be using it if it is failing). I'll include it.
Got a patch? :)
> checsktyle:check seems to fail on warning too
> ---------------------------------------------
>
> Key: MCHECKSTYLE-38
> URL: http://jira.codehaus.org/browse/MCHECKSTYLE-38
> Project: Maven 2.x Checkstyle Plugin
> Type: Bug
> Versions: 2.0
> Reporter: Vincent Massol
> Fix For: 2.1
> Attachments: checkstyle-result.xml
>
>
> failOnViolation=true is supposed to fail only on errors but it seems to be
> failing on warnings too. See attached checkstyle result file which leads to
> {noformat}
> [...]
> [INFO] [checkstyle:check {execution: default}]
> [INFO]
> ------------------------------------------------------------------------
> [ERROR] BUILD FAILURE
> [INFO]
> ------------------------------------------------------------------------
> [INFO] You have 194 checkstyle violations.
> [INFO]
> ------------------------------------------------------------------------
> {noformat}
> I think the problem is that the code checks for an <error> tag whereas it
> should check for the "severity" attribute instead.
--
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
