[commons-statistics] 01/04: Remove plugin checks from default goal

[ah...@apache.org]

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

aherbert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-statistics.git

commit 3b7c0d2cd48408f8c00034dcff315c8247a0e471
Author: Alex Herbert <ah...@apache.org>
AuthorDate: Thu Aug 19 00:25:51 2021 +0100

    Remove plugin checks from default goal
    
    These plugins already run using executions:
    
    apache-rat:check (validate)
    checkstyle:check (verify)
    
    Added to verify using executions:
    
    pmd:check
    spotbugs:check
---
 pom.xml | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/pom.xml b/pom.xml
index 4dc0a9c..d55010a 100644
--- a/pom.xml
+++ b/pom.xml
@@ -193,7 +193,8 @@
   </dependencies>
 
   <build>
-    <defaultGoal>clean verify apache-rat:check clirr:check checkstyle:check pmd:check spotbugs:check javadoc:javadoc</defaultGoal>
+    <!--  TODO: Add binary compatibility check when released. -->
+    <defaultGoal>clean verify javadoc:javadoc</defaultGoal>
     <plugins>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
@@ -296,6 +297,14 @@
             <ruleset>${statistics.parent.dir}/src/main/resources/pmd/pmd-ruleset.xml</ruleset>
           </rulesets>
         </configuration>
+        <executions>
+          <execution>
+            <phase>verify</phase>
+            <goals>
+              <goal>check</goal>
+            </goals>
+          </execution>
+        </executions>
       </plugin>
       <plugin>
         <groupId>com.github.spotbugs</groupId>
@@ -306,10 +315,19 @@
           <effort>Default</effort>
           <excludeFilterFile>${statistics.parent.dir}/src/main/resources/spotbugs/spotbugs-exclude-filter.xml</excludeFilterFile>
         </configuration>
+        <executions>
+          <execution>
+            <phase>verify</phase>
+            <goals>
+              <goal>check</goal>
+            </goals>
+          </execution>
+        </executions>
       </plugin>
       <plugin>
         <groupId>org.apache.rat</groupId>
         <artifactId>apache-rat-plugin</artifactId>
+        <!-- Note: commons-parent binds the check goal to the validate phase -->
         <configuration>
           <!--  Should agree with apache-rat-plugin config under <build> -->
           <excludes combine.children="append">