[jira] [Commented] (MCHECKSTYLE-405) The checkstyle plugin should report the number of all violations

["Jeff Thomas (Jira)" <ji...@apache.org>]

    [ 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)