pydata-sphinx-theme icon indicating copy to clipboard operation
pydata-sphinx-theme copied to clipboard

proposal: adopt precommit.ci

Open drammock opened this issue 4 months ago • 0 comments

It's a CI action that runs our pre-commit hooks in a CI action, and can be configured to auto-fix some failures. https://pre-commit.ci/

Would have saved some effort in #1711, for example. @trallard is in favor (see https://github.com/pydata/pydata-sphinx-theme/pull/1711#issuecomment-1939559894), and I'm on board too (we use it in other projects I work on).

Any objections? The main downside is that for users not super familiar with git, having a bot add commits to your PR can potentially be a bit confusing / cause subsequent pushes to fail until you've done git pull --rebase or similar. But IMO git's error messages are quite clear/instructive in that particular case, and in my experience the bot solves more problems than it creates.

drammock avatar Feb 12 '24 21:02 drammock