[ http://issues.apache.org/struts/browse/STR-2889?page=comments#action_37570 ]
James Mitchell commented on STR-2889: ------------------------------------- Patch applied, however I want to leave this open while more tweaks (if any) are made. Thanks again. > 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
