Hello Olivier, I withdraw my +1, as I think the documentation should be improved, so I would give it a +0.5 :-) (non-bindingly).
While projects worked without modifications, I now wanted to give the new functionality of "MPMD-161: PMD/CPD violation exclusions by class/issue" [1] a try, but struggle with the format of the exclusion files. At least a link in the documentation to e.g. pmd_exclude.properties[2] could be helpful and I have still not figured out whether `def.configuration.App` in there is the complete name of the class App in package def.configuration or whether I am allowed to use wildcards (sth. like *DTO for class names) in there. I would expect stuff like this to be found in an examples section or a better description of the format in the check-mojo[3]. Regards Mirko [1] https://jira.codehaus.org/browse/MPMD-161 [2] http://svn.apache.org/viewvc/maven/plugins/tags/maven-pmd-plugin-3.0/src/test/resources/unit/excludes/pmd_exclude.properties?view=markup [3] http://maven.apache.org/plugins-archives/maven-pmd-plugin-3.0/check-mojo.html#excludeFromFailureFile On Fri, Feb 8, 2013 at 4:53 PM, James Nord (jnord) <[email protected]> wrote: > +1 (non-binding) tested locally on a large multi module project. > > New violations introduced (presumably as the PMD default config has moved on). > Not checked the report syntax is still the same (aka works in Jenkins) > > >> -----Original Message----- >> From: Mirko Friedenhagen [mailto:[email protected]] >> Sent: 07 February 2013 21:06 >> To: Maven Developers List >> Subject: Re: [VOTE] Maven Pmd Plugin 3.0 (take 2) >> >> +1 (non-binding) tested with two multi-module projects (no new >> features tested and I had no violations before anyway :-)). >> >> Regards Mirko >> >> On Thu, Feb 7, 2013 at 11:00 AM, Olivier Lamy <[email protected]> wrote: >> > Hi, >> > I'd like to release Maven Pmd Plugin 3.0. >> > Note this version is based on PMD 5.0.2 (reason for the version bump). >> > We fixed 18 issues: >> > https://jira.codehaus.org/secure/IssueNavigator.jspa?reset=true&jqlQue >> > >> ry=project+%3D+MPMD+AND+fixVersion+%3D+%223.0%22+AND+status+% >> 3D+Closed >> > +ORDER+BY+priority+DESC&mode=hide >> > >> > NOTE: this version use PMD 5.0.2 which is not backwards compatible >> > with PMD 4.x (see more details here >> > http://pmd.sourceforge.net/pmd-5.0.2/) >> > >> > Staging repository: >> > https://repository.apache.org/content/repositories/maven-216/ >> > >> > Source release: >> > https://repository.apache.org/content/repositories/maven-216/org/apach >> > e/maven/plugins/maven-pmd-plugin/3.0/maven-pmd-plugin-3.0-source- >> relea >> > se.zip >> > >> > Staging site: >> > http://maven.apache.org/plugins-archives/maven-pmd-plugin-3.0/ >> > >> > Vote open for 72H >> > >> > [+1] >> > [0] >> > [-1] >> > >> > Thanks >> > >> > -- >> > Olivier Lamy >> > Talend: http://coders.talend.com >> > http://twitter.com/olamy | http://linkedin.com/in/olamy >> > >> > --------------------------------------------------------------------- >> > To unsubscribe, e-mail: [email protected] For >> > additional commands, e-mail: [email protected] >> > >> >> --------------------------------------------------------------------- >> To unsubscribe, e-mail: [email protected] For additional >> commands, e-mail: [email protected] > --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
