[ http://issues.apache.org/struts/browse/STR-2889?page=all
]
Frank W. Zammetti updated STR-2889:
-----------------------------------
Attachment: checkstyle_rules.patch
Patch for Checkstylr rules file
> 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.o
rg/mod_mbox/struts-dev/200606.mbox/%3c448F56CF.2060207omnytex.com%3e
> 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.atl
assian.com/software/jira
|