You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@commons.apache.org by gg...@apache.org on 2021/08/09 13:47:44 UTC

[commons-codec] branch master updated: Manage maven-checkstyle-plugin configuration.

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

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


The following commit(s) were added to refs/heads/master by this push:
     new f2d0d91  Manage maven-checkstyle-plugin configuration.
f2d0d91 is described below

commit f2d0d914bd02e64d7b7aa0c065f1cac71b0a1120
Author: Gary Gregory <ga...@gmail.com>
AuthorDate: Mon Aug 9 09:47:40 2021 -0400

    Manage maven-checkstyle-plugin configuration.
---
 pom.xml | 27 ++++++++++++---------------
 1 file changed, 12 insertions(+), 15 deletions(-)

diff --git a/pom.xml b/pom.xml
index e4412b0..add0a79 100644
--- a/pom.xml
+++ b/pom.xml
@@ -291,6 +291,18 @@ limitations under the License.
             </ignorePathsToDelete>
           </configuration>
         </plugin>
+        <plugin>
+          <groupId>org.apache.maven.plugins</groupId>
+          <artifactId>maven-checkstyle-plugin</artifactId>
+          <version>${checkstyle.version}</version>
+          <configuration>
+            <configLocation>${checkstyle.config.file}</configLocation>
+            <enableRulesSummary>false</enableRulesSummary>
+            <headerFile>${checkstyle.header.file}</headerFile>
+            <includeTestSourceDirectory>true</includeTestSourceDirectory>
+            <resourceExcludes>NOTICE.txt,LICENSE.txt,**/pom.properties,**/sha512.properties</resourceExcludes>
+          </configuration>
+        </plugin>        
       </plugins>
     </pluginManagement>
     <plugins>
@@ -345,14 +357,6 @@ limitations under the License.
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-checkstyle-plugin</artifactId>
-        <version>${checkstyle.version}</version>
-        <configuration>
-          <configLocation>${checkstyle.config.file}</configLocation>
-          <enableRulesSummary>false</enableRulesSummary>
-          <headerFile>${checkstyle.header.file}</headerFile>
-          <includeTestSourceDirectory>true</includeTestSourceDirectory>
-          <resourceExcludes>NOTICE.txt,LICENSE.txt,**/pom.properties,**/sha512.properties</resourceExcludes>
-        </configuration>
       </plugin>
       <!-- Specify source for JDK 11+ javadoc tool to ignore the 'unamed' module.
            See: https://bugs.openjdk.java.net/browse/JDK-8212233.
@@ -381,13 +385,6 @@ limitations under the License.
       <plugin>
         <groupId>org.apache.maven.plugins</groupId>
         <artifactId>maven-checkstyle-plugin</artifactId>
-        <version>${checkstyle.version}</version>
-        <configuration>
-          <configLocation>${checkstyle.config.file}</configLocation>
-          <enableRulesSummary>false</enableRulesSummary>
-          <headerFile>${checkstyle.header.file}</headerFile>
-          <includeTestSourceDirectory>true</includeTestSourceDirectory>
-        </configuration>
         <!-- We need to specify reportSets because 2.9.1 creates two reports -->
         <reportSets>
           <reportSet>