Skip to content

cicd(python): update pre-commit git hooks#132

Merged
jsstevenson merged 6 commits into
mainfrom
update-precommit
Dec 23, 2025
Merged

cicd(python): update pre-commit git hooks#132
jsstevenson merged 6 commits into
mainfrom
update-precommit

Conversation

@jsstevenson
Copy link
Copy Markdown
Member

@jsstevenson jsstevenson commented Dec 22, 2025

close #129
close #96

See cancervariants/gene-normalization#433 for an example

  • update actions (why not) + Python versions
  • pre-commit -> prek (idk it's faster)
  • Add some new hooks (why not)
  • Consolidate hooks into a single action run rather than exploding them in a matrix (-> much easier to review actions results)

Warning

You may have to manually clear your git hooks (rm -rf .git/hooks) in each repo, and then reinstall w/ prek install. This is kind of annoying but will only have to happen once.

@jsstevenson jsstevenson added the priority:low Low priority label Dec 22, 2025
@jsstevenson jsstevenson requested a review from a team December 23, 2025 01:49
korikuzma
korikuzma previously approved these changes Dec 23, 2025
@jsstevenson jsstevenson merged commit a7904d3 into main Dec 23, 2025
3 checks passed
@jsstevenson jsstevenson deleted the update-precommit branch December 23, 2025 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

priority:low Low priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Condense pre-commit actions to a single job Use a consistent Python version in CI actions

2 participants