From 968bd4664f6082892a3974f182202ae429006e82 Mon Sep 17 00:00:00 2001
From: Wouter Born <github@maindrain.net>
Date: Sun, 11 Aug 2024 19:59:25 +0200
Subject: [PATCH] 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>
---
 docs/maven-plugin.md                                          | 2 +-
 pom.xml                                                       | 2 +-
 .../org/openhab/tools/analysis/tools/CheckstyleChecker.java   | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/docs/maven-plugin.md b/docs/maven-plugin.md
index 10e88994..c2af63c6 100644
--- a/docs/maven-plugin.md
+++ b/docs/maven-plugin.md
@@ -101,7 +101,7 @@ Parameters:
 | ------ | ------| -------- |
 | **checkstyleRuleset** | String | Relative path of the XML configuration to use. If not set the default ruleset file will be used |
 | **checkstyleFilter** | String | Relative path of the suppressions XML file to use. If not set the default filter file will be used |
-| **maven.checkstyle.version** | String | The version of the maven-checkstyle-plugin that will be used (default value is **3.3.1**)|
+| **maven.checkstyle.version** | String | The version of the maven-checkstyle-plugin that will be used (default value is **3.4.0**)|
 | **checkstylePlugins** | List<Dependency> | A list with artifacts that contain additional checks for Checkstyle |
 | **checkstyleProperties** | String | Relative path of the properties file to use in the ruleset to configure specific checks |
 
diff --git a/pom.xml b/pom.xml
index 4c60a88f..362235d7 100644
--- a/pom.xml
+++ b/pom.xml
@@ -67,7 +67,7 @@
     <mockito.version>4.10.0</mockito.version>
     <maven.resources.version>3.3.0</maven.resources.version>
     <pmd.version>7.4.0</pmd.version>
-    <checkstyle.version>10.14.0</checkstyle.version>
+    <checkstyle.version>10.17.0</checkstyle.version>
     <spotbugs.version>4.8.6</spotbugs.version>
     <maven.core.version>3.6.0</maven.core.version>
     <maven.plugin.api.version>3.8.5</maven.plugin.api.version>
diff --git a/sat-plugin/src/main/java/org/openhab/tools/analysis/tools/CheckstyleChecker.java b/sat-plugin/src/main/java/org/openhab/tools/analysis/tools/CheckstyleChecker.java
index c1242d0a..7ab5a525 100644
--- a/sat-plugin/src/main/java/org/openhab/tools/analysis/tools/CheckstyleChecker.java
+++ b/sat-plugin/src/main/java/org/openhab/tools/analysis/tools/CheckstyleChecker.java
@@ -53,7 +53,7 @@ public class CheckstyleChecker extends AbstractChecker {
     /**
      * The version of the maven-checkstyle-plugin that will be used
      */
-    @Parameter(property = "maven.checkstyle.version", defaultValue = "3.3.1")
+    @Parameter(property = "maven.checkstyle.version", defaultValue = "3.4.0")
     private String checkstyleMavenVersion;
 
     /**
@@ -118,7 +118,7 @@ public void execute() throws MojoExecutionException {
 
         checkstylePlugins.add(dependency("org.openhab.tools.sat.custom-checks", "checkstyle", plugin.getVersion()));
         // Maven may load an older version, if no version is specified
-        checkstylePlugins.add(dependency("com.puppycrawl.tools", "checkstyle", "10.14.0"));
+        checkstylePlugins.add(dependency("com.puppycrawl.tools", "checkstyle", "10.17.0"));
         checkstylePlugins.forEach(logDependency());
 
         String baseDir = mavenProject.getBasedir().toString();