This is an automated email from the ASF dual-hosted git repository.

ggregory pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/commons-imaging.git

commit efda70b53ff923add32c53775a1d7c91c2b47560
Author: Gary Gregory <garydgreg...@gmail.com>
AuthorDate: Tue Jan 21 10:57:00 2025 -0500

    Sort checkstyle entries
    
    Remove redundant NeedBraces entry
---
 src/conf/checkstyle.xml | 43 +++++++++++++++++++++----------------------
 1 file changed, 21 insertions(+), 22 deletions(-)

diff --git a/src/conf/checkstyle.xml b/src/conf/checkstyle.xml
index 66c53b64..31730a62 100644
--- a/src/conf/checkstyle.xml
+++ b/src/conf/checkstyle.xml
@@ -22,12 +22,22 @@ limitations under the License.
 <!-- commons lang customization of default Checkstyle behavior -->
 <module name="Checker">
   <property name="localeLanguage" value="en" />
+  <module name="FileTabCharacter">
+    <property name="fileExtensions" value="java,xml" />
+  </module>  
   <module name="JavadocPackage" />
+  <module name="LineLength">
+    <property name="max" value="160" />
+  </module>
   <module name="NewlineAtEndOfFile">
     <property name="lineSeparator" value="lf" />
   </module>
-  <module name="FileTabCharacter">
-    <property name="fileExtensions" value="java,xml" />
+  <!-- @author tags are deprecated -->
+  <module name="RegexpSingleline">
+    <property name="format" value="^\s+\*\s+@author\s" />
+    <property name="message" value="Deprecated @author tag" />
+    <property name="fileExtensions" value="java" />
+    <property name="severity" value="warning" />
   </module>
   <module name="RegexpSingleline">
     <!-- \s matches whitespace character, $ matches end of line. -->
@@ -40,34 +50,23 @@ limitations under the License.
   <module name="TreeWalker">
     <module name="AvoidStarImport" />
     <module name="IllegalImport" />
-    <module name="RedundantImport" />
-    <module name="UnusedImports" />
-    <module name="NeedBraces" />
-    <!-- <module name="JavadocMethod"> <property name="allowUndeclaredRTE" 
value="true"/> <property name="scope" value="public" /> </module> -->
-    <module name="ModifierOrder" />
-    <module name="RedundantModifier" />
-    <module name="UpperEll" />
-    <module name="LeftCurly" />
-    <module name="NeedBraces" />
-    <module name="RightCurly" />
     <module name="ImportOrder">
       <property name="option" value="top" />
       <property name="groups" value="java,javax,org" />
       <property name="ordered" value="true" />
       <property name="separated" value="true" />
     </module>
+    <module name="LeftCurly" />
+    <module name="NeedBraces" />
+    <!-- <module name="JavadocMethod"> <property name="allowUndeclaredRTE" 
value="true"/> <property name="scope" value="public" /> </module> -->
+    <module name="ModifierOrder" />
+    <module name="RedundantImport" />
+    <module name="RedundantModifier" />
+    <module name="RightCurly" />
+    <module name="UpperEll" />
+    <module name="UnusedImports" />
     <module name="WhitespaceAfter" />
     <module name="WhitespaceAround" />
     <module name="WhitespaceAroundCheck" />
   </module>
-  <module name="LineLength">
-    <property name="max" value="160" />
-  </module>
-  <!-- @author tags are deprecated -->
-  <module name="RegexpSingleline">
-    <property name="format" value="^\s+\*\s+@author\s" />
-    <property name="message" value="Deprecated @author tag" />
-    <property name="fileExtensions" value="java" />
-    <property name="severity" value="warning" />
-  </module>
 </module>

Reply via email to