gian1200 commented on issue #648:
URL: 
https://github.com/apache/maven-checkstyle-plugin/issues/648#issuecomment-6159320868

   > Plain text FTW.
   
   I neither agree nor disagree; at the end of the day it's a personal choice. 
   
   To clarify the color part: I mentioned yellow only for consistency. ERRORs 
are already shown in red (and INFOs in blue), so WARNINGs would naturally be 
yellow, which matches Maven's standard level coloring. Plain text is one flag 
away for anyone who prefers it, since `-B` disables colors.
   
   --------
   
   The main point of this issue is different: it's currently not possible to 
see warnings in the logs without `consoleOutput: true`.
   
   I understand the concern about adding yet another "switch", but I couldn't 
find an existing one to reuse 
(https://maven.apache.org/plugins/maven-checkstyle-plugin/check-mojo.html).
   
   As far as I can tell, `violationSeverity=warning` would print them as 
desired, but it would also make them count as violations and fail the build, 
which is not what's wanted here.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to