Align pre-commit ruff/shellcheck hooks with CI versions#14
Merged
Conversation
The pre-commit hooks were SHA-frozen at ruff v0.8.0 and shellcheck-py v0.10.0.1, while CI (via uv.lock) runs ruff 0.15.12 and shellcheck-py 0.11.0.1 — so local `pre-commit` linted/formatted with a very different ruff than CI enforced. Dependabot's PRs #11/#12, which would have closed this gap, were auto-closed as "no longer updatable" because they were tag-based bumps that don't apply to the SHA-pinned config. Re-freeze both hooks to the commit SHAs for the versions CI already uses. Verified: `pre-commit run {ruff,ruff-format,shellcheck} --all-files` all pass at the new pins with no file changes (CI already runs these versions green against this codebase). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Re-freezes the SHA-pinned pre-commit hooks to the versions CI already runs, closing a real local/CI skew.
Background
Dependabot PRs #11 (shellcheck-py) and #12 (ruff-pre-commit) were auto-closed as "no longer updatable" — they were tag-based bumps that don't apply to the SHA-pinned
.pre-commit-config.yaml. But the gap they targeted is real:Local
pre-commitwas linting/formatting with ruff 0.8.0 while CI enforces ruff 0.15.12.Change
Re-freeze both hooks to the commit SHAs for the versions CI uses:
ruff-pre-commit→6fec9b7…(# frozen v0.15.12)shellcheck-py→745eface…(# frozen v0.11.0.1)Verification
uv run pre-commit run {ruff,ruff-format,shellcheck} --all-files— all Passed at the new pins, no file modifications (expected: CI already runs these exact versions green against this codebase).🤖 Generated with Claude Code