Uploaded image for project: 'Struts 1'
  1. Struts 1
  2. STR-2889

Updated Checkstyle rules

    XMLWordPrintableJSON

    Details

    • Flags:
      Patch

      Description

      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.

        Attachments

        1. checkstyle_rules.patch
          3 kB
          Frank W. Zammetti

          Activity

            People

            • Assignee:
              jmitchtx James Mitchell
              Reporter:
              fzammetti@omnytex.com Frank W. Zammetti
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: