A few issues: 1) The howto doc still has not been updated to explain the check goals. 2) I'd prefer the check goal to be named pmd-check for symmetry with the cpd-check goal. 3) I'd prefer the check violation statement to say "You have XX PMD|CPD violations" so it is a little more obvious why the build broke.
The latter two are quibbles. (1) means a -1 still. -----Original Message----- From: Brett Porter [mailto:[EMAIL PROTECTED] Sent: Saturday, April 08, 2006 11:26 AM To: Maven Developers List Subject: Re: [vote] Release Maven PMD plugin 2.0 Does it look ok now? Mike Perham wrote: > -1 > > The PMD and CPD check goals are not documented at all. I just checked > in a simple update to howto.apt which adds some check details. I > would like to see this fleshed out a little. > > I would also like to see http://jira.codehaus.org/browse/MPMD-24 > addressed as it basically renders the check goals useless to larger > projects (unless I'm misunderstanding how to configure the check > goals, which leads right back to the first note...) > > Six minutes within the 72 hr window! :-) > > > -----Original Message----- > From: Brett Porter [mailto:[EMAIL PROTECTED] > Sent: Saturday, April 01, 2006 10:08 PM > To: Maven Developers List > Subject: [vote] Release Maven PMD plugin 2.0 > > Based on: > maven-pmd-plugin 2.0-20060402.035815-1 (r390774) > > [ ] +1 > [ ] +0 > [ ] -1 > > http://jira.codehaus.org/secure/ReleaseNote.jspa?projectId=11140&style > Na > me=Html&version=12262 > > * [MPMD-2] - pmd has problems with package-info.java. > * [MPMD-16] - Utilise the exclusion mechanism to allow arbitrary > exclusions > * [MPMD-18] - set linkXRef to true by default, and only link if JXR > report is included to make it automatic > * [MPMD-23] - Use PMD 3.6 > * [MPMD-13] - Provide pmd:check (patch attached) > * [MPMD-14] - create pmd:cpd-check goal > > Cheers, > Brett > > > --------------------------------------------------------------------- > 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]
