chore: Remove pre-commit checkstyle in favour of maven solution

Checkstyle used in pre-commit rule depends on docker based setup, which is not intended to enforce to al developers in their environments. Using maven checkstyle solution not enforced is an intermediary solution to start improve the source quality until we get the enforce status. Signed-off-by: Helio Chissini de Castro <[email protected]>
eclipse-sw360 · Nov 5, 2024 · efcc1c0 · efcc1c0
1 parent cd8b354
commit efcc1c0
Show file tree

Hide file tree

Showing 4 changed files with 454 additions and 20 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -11,10 +11,6 @@ repos:
     hooks:
     -   id: conventional-pre-commit
         stages: [commit-msg]
--   repo: https://github.com/gherynos/pre-commit-java
-    rev: v0.6.4
-    hooks:
-    -   id: checkstyle
 -   repo: https://github.com/gitleaks/gitleaks
     rev: v8.21.2
     hooks:

diff --git a/checkstyle.xml b/checkstyle.xml
diff --git a/scripts/lint/checkstyle.xml b/scripts/lint/checkstyle.xml
@@ -0,0 +1,19 @@
+<?xml version="1.0"?>
+<!DOCTYPE module PUBLIC "-//Checkstyle//DTD Checkstyle Configuration 1.3//EN" "https://checkstyle.org/dtds/configuration_1_3.dtd">
+
+<module name="Checker">
+    <property name="charset" value="UTF-8"/>
+
+    <!-- Import Google style configuration -->
+    <module name="TreeWalker">
+        <module name="ImportControl">
+            <property name="file" value="scripts/lint/google_checks.xml"/>
+        </module>
+    </module>
+
+
+    <module name="LineLength">
+        <property name="max" value="120"/>
+        <property name="ignorePattern" value="^$"/>
+    </module>
+</module>