You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by ah...@apache.org on 2019/05/20 10:58:48 UTC

[commons-statistics] branch master updated: Added pmd plugin configuration to build.

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


The following commit(s) were added to refs/heads/master by this push:
     new f929f96  Added pmd plugin configuration to build.
f929f96 is described below

commit f929f96fe0399c08e7a545cfe9bea2ceb0fab69f
Author: aherbert <ah...@apache.org>
AuthorDate: Mon May 20 11:58:45 2019 +0100

    Added pmd plugin configuration to build.
---
 pom.xml | 58 ++++++++++++++++++++++++++++++++++++++--------------------
 1 file changed, 38 insertions(+), 20 deletions(-)

diff --git a/pom.xml b/pom.xml
index e8335d9..842ea30 100644
--- a/pom.xml
+++ b/pom.xml
@@ -176,7 +176,7 @@
   </dependencies>
 
   <build>
-    <defaultGoal>clean verify apache-rat:check clirr:check checkstyle:check spotbugs:check javadoc:javadoc</defaultGoal>
+    <defaultGoal>clean verify apache-rat:check clirr:check checkstyle:check pmd:check spotbugs:check javadoc:javadoc</defaultGoal>
     <plugins>
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
@@ -228,6 +228,12 @@
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-checkstyle-plugin</artifactId>
+        <configuration>
+          <configLocation>${statistics.parent.dir}/src/main/resources/checkstyle/checkstyle.xml</configLocation>
+          <headerLocation>${statistics.parent.dir}/src/main/resources/checkstyle/license-header.txt</headerLocation>
+          <enableRulesSummary>false</enableRulesSummary>
+          <includeResources>false</includeResources>
+        </configuration>
         <executions>
           <execution>
             <id>validate</id>
@@ -239,6 +245,18 @@
         </executions>
       </plugin>
       <plugin>
+        <artifactId>maven-pmd-plugin</artifactId>
+        <version>${statistics.pmd.version}</version>
+        <configuration>
+          <targetJdk>${maven.compiler.target}</targetJdk>
+          <skipEmptyReport>false</skipEmptyReport>
+          <analysisCache>true</analysisCache>
+          <rulesets>
+            <ruleset>${statistics.parent.dir}/src/main/resources/pmd/pmd-ruleset.xml</ruleset>
+          </rulesets>
+        </configuration>
+      </plugin>
+      <plugin>
         <groupId>com.github.spotbugs</groupId>
         <artifactId>spotbugs-maven-plugin</artifactId>
         <version>${statistics.spotbugs.version}</version>
@@ -344,16 +362,6 @@
         </reportSets>
       </plugin>
       <plugin>
-        <groupId>com.github.spotbugs</groupId>
-        <artifactId>spotbugs-maven-plugin</artifactId>
-        <version>${statistics.spotbugs.version}</version>
-        <configuration>
-          <threshold>Normal</threshold>
-          <effort>Default</effort>
-          <excludeFilterFile>${statistics.parent.dir}/src/main/resources/spotbugs/spotbugs-exclude-filter.xml</excludeFilterFile>
-        </configuration>
-      </plugin>
-      <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-checkstyle-plugin</artifactId>
         <version>${statistics.checkstyle.version}</version>
@@ -372,15 +380,6 @@
         </reportSets>
       </plugin>
       <plugin>
-        <groupId>org.codehaus.mojo</groupId>
-        <artifactId>clirr-maven-plugin</artifactId>
-        <version>${statistics.clirr.version}</version>
-        <configuration>
-          <minSeverity>${minSeverity}</minSeverity>
-          <ignoredDifferencesFile>${statistics.parent.dir}/src/main/resources/clirr/clirr-ignored.xml</ignoredDifferencesFile>
-        </configuration>
-      </plugin>
-      <plugin>
         <artifactId>maven-pmd-plugin</artifactId>
         <version>${statistics.pmd.version}</version>
         <configuration>
@@ -399,6 +398,25 @@
           </reportSet>
         </reportSets>
       </plugin>
+      <plugin>
+        <groupId>com.github.spotbugs</groupId>
+        <artifactId>spotbugs-maven-plugin</artifactId>
+        <version>${statistics.spotbugs.version}</version>
+        <configuration>
+          <threshold>Normal</threshold>
+          <effort>Default</effort>
+          <excludeFilterFile>${statistics.parent.dir}/src/main/resources/spotbugs/spotbugs-exclude-filter.xml</excludeFilterFile>
+        </configuration>
+      </plugin>
+      <plugin>
+        <groupId>org.codehaus.mojo</groupId>
+        <artifactId>clirr-maven-plugin</artifactId>
+        <version>${statistics.clirr.version}</version>
+        <configuration>
+          <minSeverity>${minSeverity}</minSeverity>
+          <ignoredDifferencesFile>${statistics.parent.dir}/src/main/resources/clirr/clirr-ignored.xml</ignoredDifferencesFile>
+        </configuration>
+      </plugin>
       <!--  MathJax -->
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>


Re: [commons-statistics] branch master updated: Added pmd plugin configuration to build.

Posted by Rob Tompkins <ch...@gmail.com>.

> On May 20, 2019, at 6:58 AM, aherbert@apache.org wrote:
> 
> 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
> 
> 
> The following commit(s) were added to refs/heads/master by this push:
>     new f929f96  Added pmd plugin configuration to build.
> f929f96 is described below
> 
> commit f929f96fe0399c08e7a545cfe9bea2ceb0fab69f
> Author: aherbert <ah...@apache.org>
> AuthorDate: Mon May 20 11:58:45 2019 +0100
> 
>    Added pmd plugin configuration to build.
> ---
> pom.xml | 58 ++++++++++++++++++++++++++++++++++++++--------------------
> 1 file changed, 38 insertions(+), 20 deletions(-)
> 
> diff --git a/pom.xml b/pom.xml
> index e8335d9..842ea30 100644
> --- a/pom.xml
> +++ b/pom.xml
> @@ -176,7 +176,7 @@
>   </dependencies>
> 
>   <build>
> -    <defaultGoal>clean verify apache-rat:check clirr:check checkstyle:check spotbugs:check javadoc:javadoc</defaultGoal>
> +    <defaultGoal>clean verify apache-rat:check clirr:check checkstyle:check pmd:check spotbugs:check javadoc:javadoc</defaultGoal>

thanks for that catch.

>     <plugins>
>       <plugin>
>         <groupId>org.apache.maven.plugins</groupId>
> @@ -228,6 +228,12 @@
>       <plugin>
>         <groupId>org.apache.maven.plugins</groupId>
>         <artifactId>maven-checkstyle-plugin</artifactId>
> +        <configuration>
> +          <configLocation>${statistics.parent.dir}/src/main/resources/checkstyle/checkstyle.xml</configLocation>
> +          <headerLocation>${statistics.parent.dir}/src/main/resources/checkstyle/license-header.txt</headerLocation>
> +          <enableRulesSummary>false</enableRulesSummary>
> +          <includeResources>false</includeResources>
> +        </configuration>
>         <executions>
>           <execution>
>             <id>validate</id>
> @@ -239,6 +245,18 @@
>         </executions>
>       </plugin>
>       <plugin>
> +        <artifactId>maven-pmd-plugin</artifactId>
> +        <version>${statistics.pmd.version}</version>
> +        <configuration>
> +          <targetJdk>${maven.compiler.target}</targetJdk>
> +          <skipEmptyReport>false</skipEmptyReport>
> +          <analysisCache>true</analysisCache>
> +          <rulesets>
> +            <ruleset>${statistics.parent.dir}/src/main/resources/pmd/pmd-ruleset.xml</ruleset>
> +          </rulesets>
> +        </configuration>
> +      </plugin>
> +      <plugin>
>         <groupId>com.github.spotbugs</groupId>
>         <artifactId>spotbugs-maven-plugin</artifactId>
>         <version>${statistics.spotbugs.version}</version>
> @@ -344,16 +362,6 @@
>         </reportSets>
>       </plugin>
>       <plugin>
> -        <groupId>com.github.spotbugs</groupId>
> -        <artifactId>spotbugs-maven-plugin</artifactId>
> -        <version>${statistics.spotbugs.version}</version>
> -        <configuration>
> -          <threshold>Normal</threshold>
> -          <effort>Default</effort>
> -          <excludeFilterFile>${statistics.parent.dir}/src/main/resources/spotbugs/spotbugs-exclude-filter.xml</excludeFilterFile>
> -        </configuration>
> -      </plugin>
> -      <plugin>
>         <groupId>org.apache.maven.plugins</groupId>
>         <artifactId>maven-checkstyle-plugin</artifactId>
>         <version>${statistics.checkstyle.version}</version>
> @@ -372,15 +380,6 @@
>         </reportSets>
>       </plugin>
>       <plugin>
> -        <groupId>org.codehaus.mojo</groupId>
> -        <artifactId>clirr-maven-plugin</artifactId>
> -        <version>${statistics.clirr.version}</version>
> -        <configuration>
> -          <minSeverity>${minSeverity}</minSeverity>
> -          <ignoredDifferencesFile>${statistics.parent.dir}/src/main/resources/clirr/clirr-ignored.xml</ignoredDifferencesFile>
> -        </configuration>
> -      </plugin>
> -      <plugin>
>         <artifactId>maven-pmd-plugin</artifactId>
>         <version>${statistics.pmd.version}</version>
>         <configuration>
> @@ -399,6 +398,25 @@
>           </reportSet>
>         </reportSets>
>       </plugin>
> +      <plugin>
> +        <groupId>com.github.spotbugs</groupId>
> +        <artifactId>spotbugs-maven-plugin</artifactId>
> +        <version>${statistics.spotbugs.version}</version>
> +        <configuration>
> +          <threshold>Normal</threshold>
> +          <effort>Default</effort>
> +          <excludeFilterFile>${statistics.parent.dir}/src/main/resources/spotbugs/spotbugs-exclude-filter.xml</excludeFilterFile>
> +        </configuration>
> +      </plugin>
> +      <plugin>
> +        <groupId>org.codehaus.mojo</groupId>
> +        <artifactId>clirr-maven-plugin</artifactId>
> +        <version>${statistics.clirr.version}</version>
> +        <configuration>
> +          <minSeverity>${minSeverity}</minSeverity>
> +          <ignoredDifferencesFile>${statistics.parent.dir}/src/main/resources/clirr/clirr-ignored.xml</ignoredDifferencesFile>
> +        </configuration>
> +      </plugin>
>       <!--  MathJax -->
>       <plugin>
>         <groupId>org.apache.maven.plugins</groupId>
> 


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@commons.apache.org
For additional commands, e-mail: dev-help@commons.apache.org