chore(deps): update pre-commit hook pre-commit/pre-commit-hooks to v6 - #116

High-level review

This PR upgrades pre-commit/pre-commit-hooks from v5.0.0 to v6.0.0 in .pre-commit-config.yaml. The change is minimal and the config does not use any hooks that were removed or changed incompatibly in v6.

Code quality and best practices

• Compatibility: v6 requires Python >= 3.9. The repo’s Python bindings already require >=3.11 (bindings/python/pyproject.toml), and GitHub runners ship with >=3.10, so CI should be fine. One small safeguard: ensure the pre-commit workflow runs with a Python >=3.9 interpreter (see below) to avoid issues for forks/self-hosted runners.
• Removed hooks in v6: check-byte-order-marker, fix-encoding-pragma. Your config does not reference these, so no action needed.
• Behavior change: file-contents-sorter disallows --unique and --ignore-case together in v6. You’re not using this hook, so no impact.

Architecture and organization

• No architectural impact; this is a straightforward dependency bump in tooling.

CI considerations

• The pre-commit workflow (.github/workflows/pre-commit.yml) uses a custom ./.github/actions/setup-environment. If that action does not explicitly install/configure Python, consider adding an actions/setup-python@v5 step (e.g., python-version: '3.11') before running pre-commit/action to guarantee Python >=3.9 is available for hook environments on all runners, including self-hosted or older images.
• pre-commit/action@v3.0.1 is compatible; no change required.

Testing

• Recommend running pre-commit locally (pre-commit run -a) and verifying the workflow passes on this branch. Given the minimal diff, failures are unlikely.

Documentation

• If any contributors are still on Python 3.8, the new hook set will fail to create environments. Consider a quick note in CONTRIBUTING.md (if present/added later) that pre-commit now requires Python >=3.9.

Conclusion

• LGTM as-is. The upgrade is safe for this repo based on the current configuration. My only actionable suggestion is to ensure the CI environment for the pre-commit job explicitly provides Python >=3.9 to avoid surprises on non-standard runners.

CodSpeed Performance Report

Merging #116 will not alter performance

_{Comparing renovate/pre-commit-pre-commit-hooks-6.x (c98fc9e) with main (357f1ea)}

Summary

✅ 5 untouched