-0.5
The checkstyle-rules-default and checkstyle-rules-strict now doesn't match anymore. ParameterNumber = 20 in default and 10 in strict Instead we should imo let ParamNumber=10 for both and enable the block exclude tokens by adding the following [1] to all our checkstyle rules <module name="SuppressionCommentFilter"/> In our code we can then just simply write > // CHECKSTYLE:OFF > doSomeDirtyStuff(); > // CHECKSTYLE:ON LieGrue, strub [1] http://checkstyle.sourceforge.net/config.html ----- Original Message ----- > From: Leonardo Uribe <[email protected]> > To: MyFaces Development <[email protected]> > Cc: > Sent: Wednesday, December 14, 2011 10:30 PM > Subject: [VOTE] release for myfaces master pom v 12 > > Hi, > > I was running the needed tasks to get the version 12 release of Apache > MyFaces Master POM. > > This release include a change to enable checkstyle plugin over myfaces core > and > an update of the developer/contributor list for myfaces site. > > Please note that this vote concerns all of the following parts: > > 1. Maven artifact group "org.apache.myfaces.myfaces" v 12 [1] > 2. Maven artifact group "org.apache.myfaces.buildtools" > checkstyle-rules v 4 > > The artifacts are deployed to a nexus staging repository [1]. > > Please take a look at the version 12 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-338 > > https://repository.apache.org/content/groups/staging/org/apache/myfaces/myfaces/12/ > > https://repository.apache.org/content/groups/staging/org/apache/myfaces/buildtools/checkstyle-rules/4/ >
