[camel] branch master updated: Do not generate the checkstyle reports but only use the check goal instead which is way faster

[gnodet]

This is an automated email from the ASF dual-hosted git repository.

gnodet pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/camel.git


The following commit(s) were added to refs/heads/master by this push:
     new 41d5056  Do not generate the checkstyle reports but only use the check 
goal instead which is way faster
41d5056 is described below

commit 41d5056776b47ee014c0e9db776c00d48567aa85
Author: Guillaume Nodet <gno...@gmail.com>
AuthorDate: Fri Jun 4 17:53:11 2021 +0200

    Do not generate the checkstyle reports but only use the check goal instead 
which is way faster
    
    The second build time goes from 117 seconds to 66 seconds using `mvnd 
-Dnoassembly -l out.txt` on my laptop
---
 camel-dependencies/pom.xml | 1 +
 parent/pom.xml             | 6 +++++-
 2 files changed, 6 insertions(+), 1 deletion(-)

diff --git a/camel-dependencies/pom.xml b/camel-dependencies/pom.xml
index c5b2fb6..7bfe1cc 100644
--- a/camel-dependencies/pom.xml
+++ b/camel-dependencies/pom.xml
@@ -120,6 +120,7 @@
     <cassandra-version>3.11.8</cassandra-version>
     <cdi-api-2.0-version>2.0</cdi-api-2.0-version>
     <cglib-version>3.2.12</cglib-version>
+    <checkstyle.failOnViolation>false</checkstyle.failOnViolation>
     <chunk-templates-version>3.6.1</chunk-templates-version>
     <citrus-version>2.8.0</citrus-version>
     <classgraph-version>4.8.52</classgraph-version>
diff --git a/parent/pom.xml b/parent/pom.xml
index 314cab9..4eab2bf 100644
--- a/parent/pom.xml
+++ b/parent/pom.xml
@@ -44,6 +44,7 @@
         <camel.surefire.parallel.factor>1</camel.surefire.parallel.factor>
         <cxf.xjc.jvmArgs/>
         <trimStackTrace>false</trimStackTrace>
+        <checkstyle.failOnViolation>false</checkstyle.failOnViolation>
         <!-- dependency versions -->
         <abdera-version>1.1.3</abdera-version>
         <!-- Note that activemq dependency is only used for testing! -->
@@ -4140,7 +4141,7 @@
                                 
<excludes>${sourcecheckExcludesComma}**/archetype-resources/**/*.java,**/archetype-resources/**/*.groovy,**/archetype-resources/**/*.scala,**/archetype-resources/**/*.xml,**/org/json/simple/**/*.java</excludes>
                             </configuration>
                             <goals>
-                                <goal>checkstyle</goal>
+                                <goal>check</goal>
                             </goals>
                         </execution>
                     </executions>
@@ -4672,6 +4673,9 @@
                     </plugin>
                 </plugins>
             </build>
+            <properties>
+                <checkstyle.failOnViolation>true</checkstyle.failOnViolation>
+            </properties>
         </profile>
 
         <profile>