Skip to content

ci: replace pre-commit/action with inlined pinned run [SEC-2493] - #131

Merged
leifdreizler merged 1 commit into
mainfrom
inline-pinned-precommit
Jul 30, 2026
Merged

ci: replace pre-commit/action with inlined pinned run [SEC-2493]#131
leifdreizler merged 1 commit into
mainfrom
inline-pinned-precommit

Conversation

@leifdreizler

@leifdreizler leifdreizler commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Removes the transitively-unpinned pre-commit/action

Similar to: https://github.com/semgrep/semgrep-rules/pull/4017/changes

Remove the transitively-unpinned pre-commit/action. Keep the existing
checkout + semgrep/setup-ocaml + ocp-indent PATH setup (the ocp-indent
hook needs the opam binary on PATH); replace only the pre-commit/action
step with inlined SHA-pinned setup-uv + cache + uv tool install
pre-commit --with pre-commit-uv (1-week cooldown) + pre-commit run.

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 20:27
@leifdreizler
leifdreizler requested a review from ajbt200128 July 29, 2026 21:13
@leifdreizler
leifdreizler enabled auto-merge (squash) July 30, 2026 22:02

@dijkstracula dijkstracula left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This does indeed seem to be an inlined version of this workflow. LGTM

@leifdreizler
leifdreizler merged commit e5b08a8 into main Jul 30, 2026
6 checks passed
@leifdreizler
leifdreizler deleted the inline-pinned-precommit branch July 30, 2026 22:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants