From 14f76b61466134f5c04378b58f95f9abd2640ce4 Mon Sep 17 00:00:00 2001 From: Ludy Date: Tue, 27 May 2025 13:17:25 +0200 Subject: [PATCH] Bump Pre-commit Hooks and Java Formatter to Latest Versions (#3589) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit # Description of Changes Please provide a summary of the changes, including: - **What was changed**: - Updated `ruff` from v0.11.6 to v0.11.11 and `gitleaks` from v8.24.3 to v8.26.0 in `.pre-commit-config.yaml` - Bumped Java formatter version from 1.26.0 to 1.27.0 in VSCode settings (`.vscode/settings.json`) and in `build.gradle` (googleJavaFormat) - Standardized quoting for the `jacoco` plugin in `build.gradle` - Cleaned up indentation and removed extra whitespace in test dependencies - **Why the change was made**: To keep our linting and formatting tools up to date with the latest stable releases—bringing in bug fixes, performance improvements, and maintaining consistency across environments. --- ## Checklist ### General - [x] I have read the [Contribution Guidelines](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/CONTRIBUTING.md) - [x] I have read the [Stirling-PDF Developer Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/DeveloperGuide.md) (if applicable) - [ ] I have read the [How to add new languages to Stirling-PDF](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/HowToAddNewLanguage.md) (if applicable) - [x] I have performed a self-review of my own code - [x] My changes generate no new warnings ### Documentation - [ ] I have updated relevant docs on [Stirling-PDF's doc repo](https://github.com/Stirling-Tools/Stirling-Tools.github.io/blob/main/docs/) (if functionality has heavily changed) - [ ] I have read the section [Add New Translation Tags](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/HowToAddNewLanguage.md#add-new-translation-tags) (for new translation tags only) ### UI Changes (if applicable) - [ ] Screenshots or videos demonstrating the UI changes are attached (e.g., as comments or direct attachments in the PR) ### Testing (if applicable) - [ ] I have tested my changes locally. Refer to the [Testing Guide](https://github.com/Stirling-Tools/Stirling-PDF/blob/main/DeveloperGuide.md#6-testing) for more details. Co-authored-by: Anthony Stirling <77850077+Frooodle@users.noreply.github.com> --- .pre-commit-config.yaml | 4 ++-- .vscode/settings.json | 2 +- build.gradle | 3 +-- 3 files changed, 4 insertions(+), 5 deletions(-) diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml index c792e50a0..beec5eb99 100644 --- a/.pre-commit-config.yaml +++ b/.pre-commit-config.yaml @@ -1,6 +1,6 @@ repos: - repo: https://github.com/astral-sh/ruff-pre-commit - rev: v0.11.6 + rev: v0.11.11 hooks: - id: ruff args: @@ -22,7 +22,7 @@ repos: files: \.(html|css|js|py|md)$ exclude: (.vscode|.devcontainer|src/main/resources|Dockerfile|.*/pdfjs.*|.*/thirdParty.*|bootstrap.*|.*\.min\..*|.*diff\.js) - repo: https://github.com/gitleaks/gitleaks - rev: v8.24.3 + rev: v8.26.0 hooks: - id: gitleaks - repo: https://github.com/pre-commit/pre-commit-hooks diff --git a/.vscode/settings.json b/.vscode/settings.json index a4be4d0cd..bfc406300 100644 --- a/.vscode/settings.json +++ b/.vscode/settings.json @@ -10,7 +10,7 @@ "java.configuration.updateBuildConfiguration": "interactive", "java.format.enabled": true, "java.format.settings.profile": "GoogleStyle", - "java.format.settings.google.version": "1.26.0", + "java.format.settings.google.version": "1.27.0", "java.format.settings.google.extra": "--aosp --skip-sorting-imports --skip-javadoc-formatting", // (DE) Aktiviert Kommentare im Java-Format. // (EN) Enables comments in Java formatting. diff --git a/build.gradle b/build.gradle index 368bee5b2..2560c991c 100644 --- a/build.gradle +++ b/build.gradle @@ -376,7 +376,7 @@ spotless { java { target project.fileTree('src').include('**/*.java') - googleJavaFormat("1.26.0").aosp().reorderImports(false) + googleJavaFormat("1.27.0").aosp().reorderImports(false) importOrder("java", "javax", "org", "com", "net", "io", "jakarta", "lombok", "me", "stirling") toggleOffOn() @@ -546,7 +546,6 @@ dependencies { // Mockito (core) testImplementation 'org.mockito:mockito-core:5.18.0' - testRuntimeOnly 'org.mockito:mockito-inline:5.2.0' }