[ 
https://jira.codehaus.org/browse/MPMD-161?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=318660#comment-318660
 ] 

Olivier Lamy commented on MPMD-161:
-----------------------------------

not really :-)
We now use mojo annotations.
Furthermore a unit or it test will be welcome for such new feature.
Thanks !
                
> PMD/CPD violation exclusions by class/issue
> -------------------------------------------
>
>                 Key: MPMD-161
>                 URL: https://jira.codehaus.org/browse/MPMD-161
>             Project: Maven 2.x PMD Plugin
>          Issue Type: New Feature
>          Components: CPD, PMD
>            Reporter: Andrey Utis
>            Assignee: Olivier Lamy
>             Fix For: 3.0
>
>         Attachments: pmd2.7.1_exclusions2.patch, pmd_exclude_trunk.patch
>
>
> I work on a system that has a lot of legacy code with a lot of PMD/CPD 
> violations. Introducing PMD to the build is not an easy option, since the 
> build would always fail. So the next best thing is to exclude any existing 
> violations and only fail if a new violation is introduced. Unfortunately, if 
> I have thousands of violations, PMD doesn't provide a good way to do this. 
> I'd have to add comments or annotations to all these places to ignore the 
> violation.
> We came up with an alternative solution to this. We modified the PMD plugin 
> to read a properties file that contains a mapping of class name to a list of 
> excluded violations. If a particular violation is in this file, it does not 
> cause the build to fail. Similar idea for CPD. 
> An svn patch is attached (apply to 2.7.1)
> (Disclaimer: this is probably not the most elegant piece of code but it works 
> and it was quick. Feel free to re-factor and make it more in line with the 
> rest of the plugin code)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://jira.codehaus.org/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to