Skip to content

Freeze pre-commit hook revisions - #791

Merged
AlexWaygood merged 1 commit into
mainfrom
alex/freeze-pre-commit-hooks
Aug 28, 2026
Merged

Freeze pre-commit hook revisions#791
AlexWaygood merged 1 commit into
mainfrom
alex/freeze-pre-commit-hooks

Conversation

@AlexWaygood

@AlexWaygood AlexWaygood commented Aug 28, 2026

Copy link
Copy Markdown
Member

Pin all seven pre-commit hook repositories to the commits corresponding to their existing versions so the hooks no longer depend on mutable tags. This is better for security. Preserve the original tags in # frozen: comments for readability.

@codecov

codecov Bot commented Aug 28, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.26%. Comparing base (a697011) to head (5df026a).

@@           Coverage Diff           @@
##             main     #791   +/-   ##
=======================================
  Coverage   97.26%   97.26%           
=======================================
  Files           3        3           
  Lines        7859     7859           
=======================================
  Hits         7644     7644           
  Misses        215      215           
Flag Coverage Δ
3.10 88.85% <ø> (ø)
3.10.4 88.85% <ø> (ø)
3.11 88.02% <ø> (ø)
3.11.0 87.27% <ø> (ø)
3.12 87.96% <ø> (ø)
3.12.0 87.94% <ø> (ø)
3.13 83.39% <ø> (ø)
3.13.0 84.09% <ø> (ø)
3.14 79.53% <ø> (ø)
3.9 89.53% <ø> (ø)
3.9.12 89.53% <ø> (ø)
pypy3.10 88.68% <ø> (ø)
pypy3.11 87.88% <ø> (ø)
pypy3.9 89.37% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AlexWaygood
AlexWaygood marked this pull request as ready for review August 28, 2026 13:27
@AlexWaygood
AlexWaygood merged commit 191303a into main Aug 28, 2026
25 checks passed
@AlexWaygood
AlexWaygood deleted the alex/freeze-pre-commit-hooks branch August 28, 2026 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants