Author: gk
Date: Wed Jan 5 14:39:17 2022
New Revision: 1896714
URL: http://svn.apache.org/viewvc?rev=1896714&view=rev
Log:
- fix & update checkstyle to v8.45.1, as checking fails and jenkins complains:
[WARNING] Old version of checkstyle detected. Consider updating to >= v8.30
and job resulted in failure.
Modified:
db/torque/trunk/pom.xml
db/torque/trunk/src/main/checkstyle/checkstyle.xml
Modified: db/torque/trunk/pom.xml
URL: http://svn.apache.org/viewvc/db/torque/trunk/pom.xml?rev=1896714&r1=1896713&r2=1896714&view=diff
==============================================================================
--- db/torque/trunk/pom.xml (original)
+++ db/torque/trunk/pom.xml Wed Jan 5 14:39:17 2022
@@ -526,6 +526,13 @@
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-checkstyle-plugin</artifactId>
<version>3.1.2</version>
+ <dependencies>
+ <dependency>
+ <groupId>com.puppycrawl.tools</groupId>
+ <artifactId>checkstyle</artifactId>
+ <version>8.45.1</version>
+ </dependency>
+ </dependencies>
</plugin>
<plugin>
<groupId>org.codehaus.mojo</groupId>
Modified: db/torque/trunk/src/main/checkstyle/checkstyle.xml
URL: http://svn.apache.org/viewvc/db/torque/trunk/src/main/checkstyle/checkstyle.xml?rev=1896714&r1=1896713&r2=1896714&view=diff
==============================================================================
--- db/torque/trunk/src/main/checkstyle/checkstyle.xml (original)
+++ db/torque/trunk/src/main/checkstyle/checkstyle.xml Wed Jan 5 14:39:17 2022
@@ -18,66 +18,71 @@
under the License.
-->
-<!DOCTYPE module PUBLIC
- "-//Puppy Crawl//DTD Check Configuration 1.2//EN"
- "http://www.puppycrawl.com/dtds/configuration_1_2.dtd">
-<!--
+<!DOCTYPE module PUBLIC
+ "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN"
+ "https://checkstyle.org/dtds/configuration_1_3.dtd">
- Checkstyle configuration that checks the sun coding conventions from:
+ <module name="Checker">
+
+ <!--
+ Checkstyle configuration that checks the sun coding conventions from:
- the Java Language Specification at
- http://java.sun.com/docs/books/jls/second_edition/html/index.html
+ https://docs.oracle.com/javase/specs/jls/se11/html/index.html
- - the Sun Code Conventions at http://java.sun.com/docs/codeconv/
+ - the Sun Code Conventions at https://www.oracle.com/java/technologies/javase/codeconventions-contents.html
- the Javadoc guidelines at
- http://java.sun.com/j2se/javadoc/writingdoccomments/index.html
+ https://www.oracle.com/technical-resources/articles/java/javadoc-tool.html
- - the JDK Api documentation http://java.sun.com/j2se/docs/api/index.html
+ - the JDK Api documentation https://docs.oracle.com/en/java/javase/11/
- some best practices
- Checkstyle is very configurable. Be sure to read the documentation at
- http://checkstyle.sf.net (or in your downloaded distribution).
-
- Most Checks are configurable, be sure to consult the documentation.
+ Checkstyle is very configurable. Be sure to read the documentation at
+ https://checkstyle.org (or in your downloaded distribution).
- To completely disable a check, just comment it out or delete it from the file.
+ Most Checks are configurable, be sure to consult the documentation.
- Finally, it is worth reading the documentation.
-
--->
+ To completely disable a check, just comment it out or delete it from the file.
+ To suppress certain violations please review suppression filters.
-<module name="Checker">
+ Finally, it is worth reading the documentation.
+ -->
+
<property name="severity" value="warning"/>
<property name="localeCountry" value="US" />
<property name="localeLanguage" value="en" />
+
+ <property name="cacheFile" value="${checkstyle.cache.file}"/>
+ <property name="tabWidth" value="4"/>
+
<module name="SuppressionFilter">
- <property name="file" value="${checkstyle.suppressions.file}"/>
+ <property name="file" value="${checkstyle.suppressions.file}"/>
</module>
- <!--
+ <!--
Checks that a package documentation exists for each package.
-->
- <!-- See http://checkstyle.sf.net/config_javadoc.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_javadoc.html -->
<module name="JavadocPackage">
<property name="allowLegacy" value="true"/>
</module>
<!-- Checks whether files end with a new line. -->
- <!-- See http://checkstyle.sf.net/config_misc.html#NewlineAtEndOfFile -->
+ <!-- See https://checkstyle.sourceforge.io/config_misc.html#NewlineAtEndOfFile -->
<module name="NewlineAtEndOfFile">
- <property name="lineSeparator" value="lf"/>
+ <property name="lineSeparator" value="lf"/>
</module>
<!-- Checks that property files contain the same keys. -->
- <!-- See http://checkstyle.sf.net/config_misc.html#Translation -->
+ <!-- See https://checkstyle.sourceforge.io/config_misc.html#Translation -->
<module name="Translation"/>
<!-- Checks for Headers -->
- <!-- See http://checkstyle.sf.net/config_header.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_header.html -->
<module name="Header">
<!-- The follow property value demonstrates the ability -->
<!-- to have access to ANT properties. In this case it uses -->
@@ -88,43 +93,42 @@
</module>
<!-- Checks for Size Violations. -->
- <!-- See http://checkstyle.sf.net/config_sizes.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_sizes.html -->
<module name="FileLength"/>
<!-- Checks for whitespace -->
- <!-- See http://checkstyle.sf.net/config_whitespace.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_whitespace.html -->
<module name="FileTabCharacter"/>
+ <module name="LineLength">
+ <property name="ignorePattern" value="^(import.*|(\*|\s)*@version.*)$"/>
+ <property name="max" value="132"/>
+ </module>
+
<module name="TreeWalker">
- <property name="cacheFile" value="${checkstyle.cache.file}"/>
-
<!-- ************************************************************** -->
<!-- Checks that are different from the sun coding conventions ones -->
<!-- ************************************************************** -->
- <property name="tabWidth" value="4"/>
+
<module name="LeftCurly">
- <property name="option" value="nl"/>
+ <property name="option" value="nl"/>
</module>
<module name="RightCurly">
- <property name="option" value="alone"/>
- </module>
- <module name="LineLength">
- <property name="ignorePattern" value="^(import.*|(\*|\s)*@version.*)$"/>
- <property name="max" value="132"/>
+ <property name="option" value="alone"/>
</module>
<module name="MemberName">
- <property name="format" value="^[a-z]+_*[a-zA-Z0-9]*$"/>
+ <property name="format" value="^[a-z]+_*[a-zA-Z0-9]*$"/>
</module>
<module name="StaticVariableName">
- <property name="format" value="^[a-z]+_*[a-zA-Z0-9]*$"/>
+ <property name="format" value="^[a-z]+_*[a-zA-Z0-9]*$"/>
</module>
<module name="ConstantName">
- <property name="format" value="^(([A-Z][A-Z0-9]*(_[A-Z0-9]+)*)|log)$"/>
+ <property name="format" value="^(([A-Z][A-Z0-9]*(_[A-Z0-9]+)*)|log)$"/>
</module>
<module name="ParameterName">
- <property name="format" value="^[a-z]+_*[a-zA-Z0-9]*$"/>
+ <property name="format" value="^[a-z]+_*[a-zA-Z0-9]*$"/>
</module>
<!-- ************************************************************** -->
@@ -132,17 +136,20 @@
<!-- ************************************************************** -->
<!-- Checks for Javadoc comments. -->
- <!-- See http://checkstyle.sf.net/config_javadoc.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_javadoc.html -->
<module name="JavadocMethod">
- <property name="excludeScope" value="private"/>
- <property name="allowUndeclaredRTE" value="true" />
</module>
+
+ <module name="JavadocStyle">
+ <property name="excludeScope" value="private"/>
+ </module>
+
<module name="JavadocType"/>
<module name="JavadocVariable"/>
<!-- Checks for Naming Conventions. -->
- <!-- See http://checkstyle.sf.net/config_naming.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_naming.html -->
<module name="LocalFinalVariableName"/>
<module name="LocalVariableName"/>
<module name="MethodName"/>
@@ -151,7 +158,7 @@
<!-- Checks for imports -->
- <!-- See http://checkstyle.sf.net/config_import.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_import.html -->
<module name="AvoidStarImport"/>
<module name="IllegalImport"/> <!-- defaults to sun.* packages -->
<module name="RedundantImport"/>
@@ -159,13 +166,13 @@
<!-- Checks for Size Violations. -->
- <!-- See http://checkstyle.sf.net/config_sizes.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_sizes.html -->
<module name="MethodLength"/>
<module name="ParameterNumber"/>
<!-- Checks for whitespace -->
- <!-- See http://checkstyle.sf.net/config_whitespace.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_whitespace.html -->
<module name="EmptyForIteratorPad"/>
<module name="NoWhitespaceAfter"/>
<module name="NoWhitespaceBefore"/>
@@ -177,55 +184,54 @@
<!-- Modifier Checks -->
- <!-- See http://checkstyle.sf.net/config_modifiers.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_modifiers.html -->
<module name="ModifierOrder"/>
<module name="RedundantModifier"/>
<!-- Checks for blocks. You know, those {}'s -->
- <!-- See http://checkstyle.sf.net/config_blocks.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_blocks.html -->
<module name="EmptyBlock"/>
<module name="NeedBraces"/>
<!-- Checks for common coding problems -->
- <!-- See http://checkstyle.sf.net/config_coding.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_coding.html -->
<module name="EmptyStatement"/>
<module name="EqualsHashCode"/>
<module name="FallThrough"/>
<module name="HiddenField">
- <property name="tokens" value="VARIABLE_DEF"/>
+ <property name="tokens" value="VARIABLE_DEF"/>
</module>
<module name="IllegalInstantiation"/>
<module name="InnerAssignment"/>
<module name="MagicNumber"/>
<module name="MissingSwitchDefault"/>
-<!-- <module name="OneMethodPrivateField" /> -->
- <module name="RedundantThrows"/>
+ <!-- <module name="OneMethodPrivateField" /> -->
+ <!-- <module name="RedundantThrows"/> -->
<module name="SimplifyBooleanExpression"/>
<module name="SimplifyBooleanReturn"/>
<!-- Checks for class design -->
- <!-- See http://checkstyle.sf.net/config_design.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_design.html -->
<module name="FinalClass"/>
<module name="HideUtilityClassConstructor"/>
<module name="VisibilityModifier">
- <module name="MissingSwitchDefault"/>
- <property name="protectedAllowed" value="true"/>
+ <property name="protectedAllowed" value="true"/>
</module>
<!-- Miscellaneous other checks. -->
- <!-- See http://checkstyle.sf.net/config_misc.html -->
+ <!-- See https://checkstyle.sourceforge.io/config_misc.html -->
<module name="ArrayTypeStyle"/>
<module name="Regexp">
- <property name="format" value="[ \t]+$"/>
- <property name="illegalPattern" value="true"/>
- <property name="message" value="Line has trailing spaces."/>
+ <property name="format" value="[ \t]+$"/>
+ <property name="illegalPattern" value="true"/>
+ <property name="message" value="Line has trailing spaces."/>
</module>
<module name="TodoComment"/>
<module name="UpperEll"/>
-
+
</module>
-</module>
+</module>
\ No newline at end of file
lmpx.com only provides a reader for public news (NNTP) servers. It is not
affiliated with the servers or forums shown here and is not responsible for
the content of articles, which is written by their respective authors.