Skip to content

ci: replace pre-commit/action with direct pre-commit run [SEC-2494] - #131

Merged
leifdreizler merged 1 commit into
developfrom
inline-pinned-precommit
Jul 27, 2026
Merged

ci: replace pre-commit/action with direct pre-commit run [SEC-2494]#131
leifdreizler merged 1 commit into
developfrom
inline-pinned-precommit

Conversation

@leifdreizler

@leifdreizler leifdreizler commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Removes the 4 transitively-unpinned pre-commit/action uses in the hook self-test harness.

Remove the 4 transitively-unpinned pre-commit/action uses in the hook
self-test harness. Install pre-commit once via SHA-pinned setup-uv +
uv tool install pre-commit --with pre-commit-uv (1-week cooldown), then
run each hook directly with pre-commit run <hook> --hook-stage manual
--files python_simple.py. Checkout, git safedir, and fake-update-file
steps are preserved.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@leifdreizler
leifdreizler marked this pull request as ready for review July 24, 2026 22:51
Comment thread .github/workflows/test-pre-commit.yml

@yosefAlsuhaibani yosefAlsuhaibani left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure!

@leifdreizler
leifdreizler merged commit 7dcd2f7 into develop Jul 27, 2026
4 checks passed
@leifdreizler
leifdreizler deleted the inline-pinned-precommit branch July 27, 2026 22:21
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