You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Wendy Smoak (JIRA)" <ji...@apache.org> on 2006/07/11 19:46:16 UTC

[jira] Resolved: (STR-2889) Updated Checkstyle rules

     [ http://issues.apache.org/struts/browse/STR-2889?page=all ]
     
Wendy Smoak resolved STR-2889:
------------------------------

    Resolution: Fixed
     Assign To: James Mitchell

http://svn.apache.org/viewvc?rev=415519&view=rev

> Updated Checkstyle rules
> ------------------------
>
>          Key: STR-2889
>          URL: http://issues.apache.org/struts/browse/STR-2889
>      Project: Struts 1
>         Type: Improvement

>   Components: Apps, Core, EL, Extras, Faces, Scripting, Taglibs, Tiles
>     Versions: 1.2 Family, 1.3 Family
>  Environment: Any
>     Reporter: Frank W. Zammetti
>     Assignee: James Mitchell
>      Fix For: 1.3.5
>  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/%3c448F56CF.2060207@omnytex.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.atlassian.com/software/jira