Return-Path: X-Original-To: apmail-maven-dev-archive@www.apache.org Delivered-To: apmail-maven-dev-archive@www.apache.org Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by minotaur.apache.org (Postfix) with SMTP id AA7B6E577 for ; Fri, 8 Feb 2013 21:38:43 +0000 (UTC) Received: (qmail 91628 invoked by uid 500); 8 Feb 2013 21:38:43 -0000 Delivered-To: apmail-maven-dev-archive@maven.apache.org Received: (qmail 91518 invoked by uid 500); 8 Feb 2013 21:38:43 -0000 Mailing-List: contact dev-help@maven.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Help: List-Post: List-Id: "Maven Developers List" Reply-To: "Maven Developers List" Delivered-To: mailing list dev@maven.apache.org Received: (qmail 91510 invoked by uid 99); 8 Feb 2013 21:38:43 -0000 Received: from minotaur.apache.org (HELO minotaur.apache.org) (140.211.11.9) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 08 Feb 2013 21:38:43 +0000 Received: from localhost (HELO mail-oa0-f46.google.com) (127.0.0.1) (smtp-auth username olamy, mechanism plain) by minotaur.apache.org (qpsmtpd/0.29) with ESMTP; Fri, 08 Feb 2013 21:38:42 +0000 Received: by mail-oa0-f46.google.com with SMTP id k1so4522963oag.5 for ; Fri, 08 Feb 2013 13:38:41 -0800 (PST) X-Received: by 10.60.32.193 with SMTP id l1mr5204675oei.114.1360359521634; Fri, 08 Feb 2013 13:38:41 -0800 (PST) MIME-Version: 1.0 Received: by 10.182.101.227 with HTTP; Fri, 8 Feb 2013 13:38:21 -0800 (PST) In-Reply-To: References: <5629A6219DB3E544AB84388D90149787F9AECA@xmb-aln-x03.cisco.com> From: Olivier Lamy Date: Fri, 8 Feb 2013 22:38:21 +0100 Message-ID: Subject: Re: [VOTE] Maven Pmd Plugin 3.0 (take 2) To: Maven Developers List Content-Type: text/plain; charset=ISO-8859-1 I agree. doc page added in trunk and site redeployed http://maven.apache.org/plugins-archives/maven-pmd-plugin-3.0/examples/violation-exclusions.html As it's only a documentation page we usually don't retag/redeploy/etc.. a release for that. 2013/2/8 Mirko Friedenhagen : > 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) 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:mfriedenhagen@gmail.com] >>> 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 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: dev-unsubscribe@maven.apache.org For >>> > additional commands, e-mail: dev-help@maven.apache.org >>> > >>> >>> --------------------------------------------------------------------- >>> To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org For additional >>> commands, e-mail: dev-help@maven.apache.org >> > > --------------------------------------------------------------------- > To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org > For additional commands, e-mail: dev-help@maven.apache.org > -- Olivier Lamy Talend: http://coders.talend.com http://twitter.com/olamy | http://linkedin.com/in/olamy --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscribe@maven.apache.org For additional commands, e-mail: dev-help@maven.apache.org