Skip to content

ci: switch Pre-commit Checks workflow from pre-commit to prek#854

Merged
potiuk merged 1 commit into
mainfrom
ci-prek
May 19, 2026
Merged

ci: switch Pre-commit Checks workflow from pre-commit to prek#854
potiuk merged 1 commit into
mainfrom
ci-prek

Conversation

@potiuk
Copy link
Copy Markdown
Member

@potiuk potiuk commented May 18, 2026

Summary

AGENTS.md (since the prek migration) tells contributors to use prek locally and states "the hooks are the same ones CI enforces", but the Pre-commit Checks workflow was still installing pre-commit via uv tool install pre-commit. Swap both lines so CI matches the local dev path: install prek via uv, then prek run --all-files.

Same hooks (prek reads .pre-commit-config.yaml verbatim), same install transport (uv tool install …), just the faster Rust-native runner end-to-end.

Diff

-          uv tool install pre-commit
+          uv tool install prek
           uv tool ensurepath
-          pre-commit run --all-files --color always --show-diff-on-failure
+          prek run --all-files --color always --show-diff-on-failure

Test plan

  • CI's pre-commit job (renamed contents, same job name for branch-protection compatibility) runs prek and matches what prek run --all-files produces locally.

AGENTS.md (since the prek migration) tells contributors to use prek
locally and states "the hooks are the same ones CI enforces", but the
Pre-commit Checks workflow was still installing `pre-commit` via
`uv tool install pre-commit`. Swap both lines so CI matches the local
dev path: install prek via uv, then `prek run --all-files`.

Same hooks (prek reads .pre-commit-config.yaml verbatim) and same
install transport (`uv tool install …`), just the faster Rust-native
runner end-to-end.

Generated-by: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@potiuk potiuk requested review from dfoulks1 and ppkarwasz as code owners May 18, 2026 22:59
@potiuk potiuk merged commit 976304a into main May 19, 2026
8 checks passed
@potiuk potiuk deleted the ci-prek branch May 19, 2026 08:08
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