Skip to content

Comments

ci: migrate from pre-commit to prek#1564

Merged
georgeh0 merged 10 commits intococoindex-io:mainfrom
Haleshot:haleshot/migrate-precommit-prek
Jan 27, 2026
Merged

ci: migrate from pre-commit to prek#1564
georgeh0 merged 10 commits intococoindex-io:mainfrom
Haleshot:haleshot/migrate-precommit-prek

Conversation

@Haleshot
Copy link
Contributor

@Haleshot Haleshot commented Jan 26, 2026

Closes #1552

Swaps out pre-commit for prek.

prek is a single binary with no Python deps, so we can ditch the virtualenv stuff (and all those related deps like cfgv, nodeenv, virtualenv, etc.; read the lockfile diff for reference). CI gets cleaner too since prek-action simplifies setup to a single step; no more separate cache and install steps.

Also added a Dependabot entry for the pre-commit ecosystem (yes, pre-commit — Dependabot reads the same config format) for hook versions to update automatically.

Everything else is just s/pre-commit/prek/ in the docs and SKIPPREK_SKIP in the workflows.

Copy link
Contributor Author

@Haleshot Haleshot Jan 26, 2026

Choose a reason for hiding this comment

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

Dependabot's pre-commit ecosystem works w/ prek since both read the same .pre-commit-config.yaml format. Changed from monthly (the old autoupdate_schedule in the ci block) to weekly to match the github-actions update cadence.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Apologies for the oversight; I thought Dependabot had a pre-commit ecosystem, but after checking the Dependabot options reference | GitHub docs (prompted by the failed CI run), turns out it doesn't.

Will remove this entry in a follow-up; for hook version updates we'd need to run prek autoupdate manually or set up a scheduled action.

Copy link
Member

@georgeh0 georgeh0 left a comment

Choose a reason for hiding this comment

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

Thanks for making the migration happen smoothly!

@georgeh0 georgeh0 merged commit 004fe13 into cocoindex-io:main Jan 27, 2026
13 checks passed
j178 pushed a commit to j178/prek that referenced this pull request Jan 28, 2026
@Haleshot Haleshot deleted the haleshot/migrate-precommit-prek branch January 28, 2026 03:25
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.

[FEATURE] Consider migrating from pre-commit to prek

2 participants