[ 
https://issues.apache.org/jira/browse/MCHECKSTYLE-405?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17629405#comment-17629405
 ] 

Jeff Thomas commented on MCHECKSTYLE-405:
-----------------------------------------

Came here looking for this exact issue :)

I also find the reported number of violations '0' when 
"{{{}failOnViolation=false{}}}" misleading. 

I am a wee bit OCD so I want the count even if I don't want to break the build 
on violations. :P 

> The checkstyle plugin should report the number of all violations
> ----------------------------------------------------------------
>
>                 Key: MCHECKSTYLE-405
>                 URL: https://issues.apache.org/jira/browse/MCHECKSTYLE-405
>             Project: Maven Checkstyle Plugin
>          Issue Type: Improvement
>            Reporter: Guillaume Nodet
>            Priority: Major
>
> When there is an error, the build looks like:
> {code:java}
> [INFO] There is 1 error reported by Checkstyle 8.41.1 with 
> config/maven_checks.xml ruleset.
> [ERROR] 
> src/main/java/org/apache/maven/execution/BuildResumptionDataRepository.java:[54,30]
>  (whitespace) ParenPad: '(' is not followed by whitespace.
>  {code}
> and when this error is fixed, the build then looks like:
> {code}
> [INFO] You have 0 Checkstyle violations.
> {code}
> The problem is that checkstyle never caches the results for files with 
> violations, even at info or warning level.  This cause the files to be always 
> processed, without the user really being aware of any problem.
> The number of violations at info or warning level should also be printed.  In 
> addition, if there are errors, the output should be logged at WARN level.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to