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>