Skip to content

Commit bdd2fa6

Browse files
author
Niall Pemberton
committed
COMMONSSITE-30 - fix the checkstyle configuration to work for multi-module and review/update <reporting> plugin version numbers for the poms being changed.
- pom's need to use ${basedir} for configuration files as the multi-module profile is run from commons-parent, so if not specified it can't find those files - the checkstyle configuration file needs to use ${checkstyle.header.file} for the headerFile property and <headerFile> element added to the plugin config in the pom.xml - fix m1 checkstyle config to reflect the above where appropriate (specify header file property) git-svn-id: https://svn.apache.org/repos/asf/commons/sandbox/csv/trunk@659747 13f79535-47bb-0310-9956-ffa450edef68
1 parent 0888f8a commit bdd2fa6

1 file changed

Lines changed: 4 additions & 2 deletions

File tree

pom.xml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@
7373
<plugin>
7474
<groupId>org.apache.maven.plugins</groupId>
7575
<artifactId>maven-changes-plugin</artifactId>
76-
<version>2.0-beta-2</version>
76+
<version>2.0</version>
7777
<configuration>
7878
<issueLinkTemplate>%URL%/%ISSUE%</issueLinkTemplate>
7979
</configuration>
@@ -88,8 +88,10 @@
8888
<plugin>
8989
<groupId>org.apache.maven.plugins</groupId>
9090
<artifactId>maven-checkstyle-plugin</artifactId>
91+
<version>2.1</version>
9192
<configuration>
92-
<configLocation>checkstyle.xml</configLocation>
93+
<configLocation>${basedir}/checkstyle.xml</configLocation>
94+
<enableRulesSummary>false</enableRulesSummary>
9395
</configuration>
9496
</plugin>
9597
</plugins>

0 commit comments

Comments
 (0)