[ http://issues.apache.org/struts/browse/STR-2889?page=comments#action_37571 ]
Wendy Smoak commented on STR-2889: ---------------------------------- I deployed a snapshot of struts-build.jar, including Frank's changes. * http://people.apache.org/maven-snapshot-repository/org/apache/struts/struts-build/ To see the new rules in effect, change the struts-build <extension> version in the struts-action-parent (or other) pom from v1.0.1 to v1.0.2-SNAPSHOT. (To try out other rule changes locally, modify maven/build/struts_checks.xml, then 'mvn install' from there. The maven/build directory needs to be organized into src/main/resources, eventually.) > Updated Checkstyle rules > ------------------------ > > Key: STR-2889 > URL: http://issues.apache.org/struts/browse/STR-2889 > Project: Struts Action 1 > Type: Improvement > Components: Core, Apps, EL, Extras, Faces, Scripting, Taglibs, Tiles > Versions: 1.2 Family, 1.3 Family > Environment: Any > Reporter: Frank W. Zammetti > Attachments: checkstyle_rules.patch > > Argh, I have the priority choices... shouldn't there be something in between > minor and major?!? But I digress... > With reference to this thread on the @dev list: > http://mail-archives.apache.org/mod_mbox/struts-dev/200606.mbox/[EMAIL > PROTECTED] > Attached is a patch for the Checkstyle rules that eliminates the two things > discussed in this thread. It resolves a few hundred exceptions in core alone > (350-400, give or take), and should do the same for all other subprojects. > There seemed to be a feeling (don't want to say consensus at this point!) > that applying this before the next milestone would be acceptable. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/struts/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira
