Skip to content

Commit 968bd46

Browse files
authored
Upgrade Checkstyle to 10.17.0 (#469)
Upgrades Checkstyle from 10.14.0 to 10.17.0. For release notes, see: https://checkstyle.sourceforge.io/releasenotes.html#Release_10.17.0 Signed-off-by: Wouter Born <github@maindrain.net>
1 parent 2dfdbed commit 968bd46

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

docs/maven-plugin.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,7 @@ Parameters:
101101
| ------ | ------| -------- |
102102
| **checkstyleRuleset** | String | Relative path of the XML configuration to use. If not set the default ruleset file will be used |
103103
| **checkstyleFilter** | String | Relative path of the suppressions XML file to use. If not set the default filter file will be used |
104-
| **maven.checkstyle.version** | String | The version of the maven-checkstyle-plugin that will be used (default value is **3.3.1**)|
104+
| **maven.checkstyle.version** | String | The version of the maven-checkstyle-plugin that will be used (default value is **3.4.0**)|
105105
| **checkstylePlugins** | List<Dependency> | A list with artifacts that contain additional checks for Checkstyle |
106106
| **checkstyleProperties** | String | Relative path of the properties file to use in the ruleset to configure specific checks |
107107

pom.xml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@
6767
<mockito.version>4.10.0</mockito.version>
6868
<maven.resources.version>3.3.0</maven.resources.version>
6969
<pmd.version>7.4.0</pmd.version>
70-
<checkstyle.version>10.14.0</checkstyle.version>
70+
<checkstyle.version>10.17.0</checkstyle.version>
7171
<spotbugs.version>4.8.6</spotbugs.version>
7272
<maven.core.version>3.6.0</maven.core.version>
7373
<maven.plugin.api.version>3.8.5</maven.plugin.api.version>

sat-plugin/src/main/java/org/openhab/tools/analysis/tools/CheckstyleChecker.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public class CheckstyleChecker extends AbstractChecker {
5353
/**
5454
* The version of the maven-checkstyle-plugin that will be used
5555
*/
56-
@Parameter(property = "maven.checkstyle.version", defaultValue = "3.3.1")
56+
@Parameter(property = "maven.checkstyle.version", defaultValue = "3.4.0")
5757
private String checkstyleMavenVersion;
5858

5959
/**
@@ -118,7 +118,7 @@ public void execute() throws MojoExecutionException {
118118

119119
checkstylePlugins.add(dependency("org.openhab.tools.sat.custom-checks", "checkstyle", plugin.getVersion()));
120120
// Maven may load an older version, if no version is specified
121-
checkstylePlugins.add(dependency("com.puppycrawl.tools", "checkstyle", "10.14.0"));
121+
checkstylePlugins.add(dependency("com.puppycrawl.tools", "checkstyle", "10.17.0"));
122122
checkstylePlugins.forEach(logDependency());
123123

124124
String baseDir = mavenProject.getBasedir().toString();

0 commit comments

Comments
 (0)