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

[davsclaus]

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

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


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

commit 3478d8899708549e3db4916e0b1792c9b85f1e8e
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 2cf0f08..7daac8a 100644
--- a/camel-dependencies/pom.xml
+++ b/camel-dependencies/pom.xml
@@ -115,6 +115,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.107</classgraph-version>
diff --git a/parent/pom.xml b/parent/pom.xml
index 5c1abed..7d59ce6 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! -->
@@ -4179,7 +4180,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>
@@ -4711,6 +4712,9 @@
                     </plugin>
                 </plugins>
             </build>
+            <properties>
+                <checkstyle.failOnViolation>true</checkstyle.failOnViolation>
+            </properties>
         </profile>
 
         <profile>