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

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

commit 1238424c9d65eaa435fbf86c7920245b2876e84e
Author: Alex Herbert <[email protected]>
AuthorDate: Sat Sep 24 14:42:10 2022 +0100

    Update checkstyle config using latest example
    
    Updated from:
    
https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/sun_checks.xml
---
 src/main/resources/checkstyle/checkstyle.xml | 51 ++++++++++++++--------------
 1 file changed, 26 insertions(+), 25 deletions(-)

diff --git a/src/main/resources/checkstyle/checkstyle.xml 
b/src/main/resources/checkstyle/checkstyle.xml
index b3bf596b..51b420ae 100644
--- a/src/main/resources/checkstyle/checkstyle.xml
+++ b/src/main/resources/checkstyle/checkstyle.xml
@@ -26,6 +26,15 @@
   
https://github.com/checkstyle/checkstyle/blob/master/src/main/resources/sun_checks.xml
 -->
 <module name="Checker">
+  <property name="fileExtensions" value="java, properties, xml" />
+
+  <!-- Excludes all 'module-info.java' files              -->
+  <!-- See https://checkstyle.org/config_filefilters.html -->
+  <module name="BeforeExecutionExclusionFileFilter">
+    <property name="fileNamePattern" value="module\-info\.java$"/>
+  </module>
+
+  <!-- https://checkstyle.org/config_filters.html#SuppressionFilter -->
   <module name="SuppressionFilter">
     <!-- Default property set by maven-checkstyle-plugin -->
     <property name="file" value="${checkstyle.suppressions.file}"/>
@@ -34,28 +43,20 @@
 
   <property name="localeLanguage" value="en"/>
 
-  <property name="fileExtensions" value="java, properties, xml" />
-
-  <!-- Excludes all 'module-info.java' files -->
-  <!-- See https://checkstyle.org/config_filefilters.html -->
-  <module name="BeforeExecutionExclusionFileFilter">
-    <property name="fileNamePattern" value="module\-info\.java$" />
-  </module>
-
   <!-- Checks that a package-info.java file exists for each package. -->
-  <!-- See 
http://checkstyle.sourceforge.net/config_javadoc.html#JavadocPackage -->
+  <!-- See https://checkstyle.org/config_javadoc.html#JavadocPackage -->
   <module name="JavadocPackage" />
 
   <!-- Checks whether files end with a new line. -->
-  <!-- See 
http://checkstyle.sourceforge.net/config_misc.html#NewlineAtEndOfFile -->
+  <!-- See https://checkstyle.org/config_misc.html#NewlineAtEndOfFile -->
   <module name="NewlineAtEndOfFile" />
 
   <!-- Checks that property files contain the same keys. -->
-  <!-- See http://checkstyle.sourceforge.net/config_misc.html#Translation -->
+  <!-- See https://checkstyle.org/config_misc.html#Translation -->
   <module name="Translation" />
 
   <!-- Checks for Size Violations. -->
-  <!-- See http://checkstyle.sourceforge.net/config_sizes.html -->
+  <!-- See https://checkstyle.org/config_sizes.html -->
   <module name="FileLength" />
   <module name="LineLength">
     <property name="fileExtensions" value="java"/>
@@ -65,11 +66,11 @@
   </module>
 
   <!-- Checks for whitespace -->
-  <!-- See http://checkstyle.sourceforge.net/config_whitespace.html -->
+  <!-- See http://checkstyle.org/config_whitespace.html -->
   <module name="FileTabCharacter" />
 
   <!-- Miscellaneous other checks. -->
-  <!-- See http://checkstyle.sourceforge.net/config_misc.html -->
+  <!-- See http://checkstyle.org/config_misc.html -->
   <module name="RegexpSingleline">
     <property name="format" value="\s+$" />
     <property name="minimum" value="0" />
@@ -78,7 +79,7 @@
   </module>
 
   <!-- Checks for Headers -->
-  <!-- See http://checkstyle.sourceforge.net/config_header.html -->
+  <!-- See http://checkstyle.org/config_header.html -->
   <module name="Header">
     <property name="headerFile" value="${checkstyle.header.file}"/>
   </module>
@@ -86,7 +87,7 @@
   <module name="TreeWalker">
 
     <!-- Checks for Javadoc comments. -->
-    <!-- See http://checkstyle.sourceforge.net/config_javadoc.html -->
+    <!-- See http://checkstyle.org/config_javadoc.html -->
     <module name="InvalidJavadocPosition"/>
     <module name="JavadocMethod" />
     <module name="JavadocType" />
@@ -101,7 +102,7 @@
     </module>
 
     <!-- Checks for Naming Conventions. -->
-    <!-- See http://checkstyle.sourceforge.net/config_naming.html -->
+    <!-- See http://checkstyle.org/config_naming.html -->
     <module name="ConstantName" />
     <module name="LocalFinalVariableName" />
     <module name="LocalVariableName" />
@@ -113,7 +114,7 @@
     <module name="TypeName" />
 
     <!-- Checks for imports -->
-    <!-- See http://checkstyle.sourceforge.net/config_import.html -->
+    <!-- See http://checkstyle.org/config_import.html -->
     <module name="AvoidStarImport" />
     <module name="IllegalImport" /> <!-- defaults to sun.* packages -->
     <module name="RedundantImport" />
@@ -122,12 +123,12 @@
     </module>
 
     <!-- Checks for Size Violations. -->
-    <!-- See http://checkstyle.sourceforge.net/config_sizes.html -->
+    <!-- See http://checkstyle.org/config_sizes.html -->
     <module name="MethodLength" />
     <module name="ParameterNumber" />
 
     <!-- Checks for whitespace -->
-    <!-- See http://checkstyle.sourceforge.net/config_whitespace.html -->
+    <!-- See http://checkstyle.org/config_whitespace.html -->
     <module name="EmptyForIteratorPad" />
     <module name="GenericWhitespace" />
     <module name="MethodParamPad" />
@@ -148,12 +149,12 @@
     </module>
 
     <!-- Modifier Checks -->
-    <!-- See http://checkstyle.sourceforge.net/config_modifiers.html -->
+    <!-- See http://checkstyle.org/config_modifiers.html -->
     <module name="ModifierOrder" />
     <module name="RedundantModifier" />
 
     <!-- Checks for blocks. You know, those {}'s -->
-    <!-- See http://checkstyle.sourceforge.net/config_blocks.html -->
+    <!-- See http://checkstyle.org/config_blocks.html -->
     <module name="AvoidNestedBlocks" />
     <module name="EmptyBlock" />
     <module name="LeftCurly" />
@@ -161,7 +162,7 @@
     <module name="RightCurly" />
 
     <!-- Checks for common coding problems -->
-    <!-- See http://checkstyle.sourceforge.net/config_coding.html -->
+    <!-- See http://checkstyle.org/config_coding.html -->
     <module name="EmptyStatement" />
     <module name="EqualsHashCode" />
     <!-- Method parameters and local variables should not hide fields, except 
in constructors and setters -->
@@ -182,7 +183,7 @@
     <module name="SimplifyBooleanReturn" />
 
     <!-- Checks for class design -->
-    <!-- See http://checkstyle.sourceforge.net/config_design.html -->
+    <!-- See http://checkstyle.org/config_design.html -->
     <module name="DesignForExtension" />
     <module name="FinalClass" />
     <module name="HideUtilityClassConstructor" />
@@ -193,7 +194,7 @@
     </module>
 
     <!-- Miscellaneous other checks. -->
-    <!-- See http://checkstyle.sourceforge.net/config_misc.html -->
+    <!-- See http://checkstyle.org/config_misc.html -->
     <module name="ArrayTypeStyle" />
     <!-- <module name="FinalParameters" />  -->
     <module name="TodoComment">

Reply via email to