Update pmd configuration to use dogfood and fail build on violations

Refs #361
This commit is contained in:
Andreas Dangel
2018-02-11 10:57:36 +01:00
parent 0967d50275
commit e9abdf7a92

12
pom.xml
View File

@ -515,7 +515,7 @@ Additionally it includes CPD, the copy-paste-detector. CPD finds duplicated code
<execution> <execution>
<phase>verify</phase> <phase>verify</phase>
<goals> <goals>
<goal>pmd</goal> <goal>check</goal>
<goal>cpd</goal> <goal>cpd</goal>
</goals> </goals>
</execution> </execution>
@ -524,12 +524,14 @@ Additionally it includes CPD, the copy-paste-detector. CPD finds duplicated code
<minimumTokens>100</minimumTokens> <minimumTokens>100</minimumTokens>
<targetJdk>1.${java.version}</targetJdk> <targetJdk>1.${java.version}</targetJdk>
<rulesets> <rulesets>
<ruleset>${pmd.dogfood.ruleset}</ruleset> <ruleset>/net/sourceforge/pmd/pmd-dogfood-config.xml</ruleset>
</rulesets> </rulesets>
<excludeRoots> <excludeRoots>
<excludeRoot>target/generated-sources/javacc</excludeRoot> <excludeRoot>target/generated-sources/javacc</excludeRoot>
</excludeRoots> </excludeRoots>
<skipPmdError>false</skipPmdError> <skipPmdError>false</skipPmdError>
<failOnViolation>true</failOnViolation>
<printFailingErrors>true</printFailingErrors>
</configuration> </configuration>
<dependencies> <dependencies>
<!-- Note: we can't use a property for the version here due to https://issues.apache.org/jira/browse/MRELEASE-932 --> <!-- Note: we can't use a property for the version here due to https://issues.apache.org/jira/browse/MRELEASE-932 -->
@ -543,6 +545,12 @@ Additionally it includes CPD, the copy-paste-detector. CPD finds duplicated code
<artifactId>pmd-java</artifactId> <artifactId>pmd-java</artifactId>
<version>6.0.1</version> <version>6.0.1</version>
</dependency> </dependency>
<!-- This contains the dogfood ruleset -->
<dependency>
<groupId>net.sourceforge.pmd</groupId>
<artifactId>pmd-build-tools-config</artifactId>
<version>1.1.0-SNAPSHOT</version>
</dependency>
</dependencies> </dependencies>
</plugin> </plugin>
<plugin> <plugin>