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

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|

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

Rob Tompkins


> On May 20, 2019, at 6:58 AM, [hidden email] 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 <[hidden email]>
> 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: [hidden email]
For additional commands, e-mail: [hidden email]