If we only do this for the checkstyle stuff, then I'm -1, otherwise I'm +1 (don't like to block other stuff)
Reason: the checkstyle rules still miss the SuppressionCommentFilter and the rules are pretty much blown up. This is partly my fault (didn't find time to fix that), but will do now. The idea is do enable the SuppressionCommentFilter in the checkstyle rules and use // CHECKSTYLE:OFF dosomeDirtyStuff() // CHECKSTYLE:ON to partly disable the checkstyle verification on a local area. Instead of this we currently have <!-- In MyFaces Core we need methods with more than 10 parameters, for example when rendering client behaviors. --> <module name="ParameterNumber"> <property name="max" value="20" /> </module> which is really only needed for 1 method I think ;) Will move this back to max 10 and add the comment stuff. LieGrue, strub ----- Original Message ----- > From: Martin Koci <[email protected]> > To: MyFaces Development <[email protected]> > Cc: > Sent: Monday, January 23, 2012 7:38 PM > Subject: Re: [VOTE] release for myfaces master pom v 13 > > +1 > > > Leonardo Uribe píše v St 18. 01. 2012 v 09:18 -0500: >> Hi, >> >> I was running the needed tasks to get the version 13 release of Apache >> MyFaces Master POM. >> >> This release is necessary to update the site information about MyFaces >> contributors. >> >> Please note that this vote concerns all of the following parts: >> >> 1. Maven artifact group "org.apache.myfaces.myfaces" v 13 [1] >> >> The artifacts are deployed to a nexus staging repository [1]. >> >> Please take a look at the version 13 pom and vote! >> >> Please note: This vote is "majority approval" with a minimum of > three >> +1 votes (see [3]). >> >> ------------------------------------------------ >> [ ] +1 for community members who have reviewed the bits >> [ ] +0 >> [ ] -1 for fatal flaws that should cause these bits not to be released, >> and why.............. >> ------------------------------------------------ >> >> Thanks, >> Leonardo Uribe >> >> [1] > https://repository.apache.org/content/repositories/orgapachemyfaces-087/ >> > https://repository.apache.org/content/groups/staging/org/apache/myfaces/myfaces/13/ >
