Skip to content

Commit

Permalink
Exclude generated sources from PMD reports.
Browse files Browse the repository at this point in the history
  • Loading branch information
@johnmay
johnmay committed Oct 1, 2014
1 parent 52baf0c commit b885009
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions pom.xml
View file
Expand Up @@ -568,6 +568,9 @@
<rulesets>
<ruleset>/pmd/custom.xml</ruleset>
</rulesets>
<excludeRoots>
<excludeRoot>target/generated-sources/</excludeRoot>
</excludeRoots>
</configuration>
<dependencies>
<dependency>
Expand Down

4 comments on commit b885009

@egonw
Copy link
Member

@egonw egonw commented on b885009 Oct 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good thinking!

@johnmay
Copy link
Member Author

@johnmay johnmay commented on b885009 Oct 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Met with Mark yesterday at UK QSAR and looking through the PMD reports he noted that most of the "Unused.." were now in the javacc code.

@egonw
Copy link
Member

@egonw egonw commented on b885009 Oct 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, I had seen that it was a substantial part, but had not realized it would be easy in Maven to exclude it :)

(I can see other roots to be excluded...)

@johnmay
Copy link
Member Author

@johnmay johnmay commented on b885009 Oct 1, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm doesn't seem to be picked up by maven though... trying a clean build.

Please sign in to comment.