Skip to content

[CI] Add pre-commit workflow with 10+ hooks#294

Open
jbampton wants to merge 2 commits intoapache:masterfrom
jbampton:add-pre-commit
Open

[CI] Add pre-commit workflow with 10+ hooks#294
jbampton wants to merge 2 commits intoapache:masterfrom
jbampton:add-pre-commit

Conversation

@jbampton
Copy link
Member

@sebbASF
Copy link
Contributor

sebbASF commented Mar 16, 2026

I have some questions:

  • is the action approved by Infra?
  • could the checks be applied to PRs?

@jbampton
Copy link
Member Author

Here is another example of a "pre-commit" workflow:

https://github.com/apache/tooling-trusted-releases/blob/main/.github/workflows/analyze.yml

They use "prek" and uv and pin hashes. We could use this format if needed.

https://github.com/j178/prek

When GitHub actions are first added to a repository via a PR they don't run for security reasons.

So after this PR is merged they should run on subsequent PRs.

cc @dave2wave

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