Skip to content

ci: switch from pre-commit to prek NOTASK#38

Merged
gavinelder merged 1 commit into
mainfrom
ci/switch-pre-commit-to-prek
Mar 25, 2026
Merged

ci: switch from pre-commit to prek NOTASK#38
gavinelder merged 1 commit into
mainfrom
ci/switch-pre-commit-to-prek

Conversation

@gavinelder
Copy link
Copy Markdown
Contributor

Summary

  • Replace pre-commit/action with j178/prek-action, a fast Rust-based drop-in replacement
  • Remove the actions/setup-python step since prek doesn't need Python
  • Resolves the issue where pre-commit/action internally uses actions/cache@v4 as a tag reference instead of a SHA-pinned commit, violating org policy

Reference: https://github.com/seqeralabs/fusion/pull/1280

Test plan

  • Verify the pre-commit workflow passes on this PR
  • Confirm prek runs the same hooks from .pre-commit-config.yaml

🤖 Generated with Claude Code

Replace pre-commit/action with j178/prek-action, a fast Rust-based
drop-in replacement. This removes the setup-python step (prek doesn't
need Python) and resolves the issue where pre-commit/action internally
uses actions/cache@v4 as a tag reference instead of a SHA-pinned commit,
violating org policy.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 25, 2026

🐳 Docker PR Build

Status ✅ Built successfully
Commit 9ce2b292f00cb8667c34421bc9b198c821d65699
Standard ghcr.io/seqeralabs/node-red-seqera:pr-38
Studios ghcr.io/seqeralabs/node-red-seqera-studios:pr-38

View workflow run

@gavinelder gavinelder requested a review from ewels March 25, 2026 14:23
@gavinelder gavinelder merged commit 5db7124 into main Mar 25, 2026
9 checks passed
@gavinelder gavinelder deleted the ci/switch-pre-commit-to-prek branch March 25, 2026 14:58
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