> On 9 Nov 2019, at 12:23, Gilles Sadowski <gillese...@gmail.com> wrote:
> 
>> Snip
> 
> I think that I tried
>  $ mvn -Dcheckstyle.failOnViolation=false test
> 
> And still it wouldn't run the test (because I had introduced the
> "System.out" forbidden construct).

Checkstyle is configured to run in the validate phase (right at the start) [1]. 
The default is normally verify. This was copied from commons RNG and predates 
my joining the project.

So this is why you cannot run ‘mvn test’ as it runs checkstyle:check and fails 
you before doing the test.

Now fail on violation is ‘true’ perhaps we should move the check back to 
validate. So you cannot do a 'mvn install' with any checkstyle errors. You can 
also run mvn checkstyle:check to run the goal manually.

[1] http://maven.apache.org/ref/3.6.2/maven-core/lifecycles.html 
<http://maven.apache.org/ref/3.6.2/maven-core/lifecycles.html>

Reply via email to