Please use pre-commit.ci insead of the pre-commit action #2140
Replies: 2 comments
|
@CodiumAI-Agent /improve |
|
Thanks for the suggestion, and apologies it sat so long. The situation has moved since you filed it: pre-commit now runs on pull requests, so the hooks are enforced in CI rather than being manual-only, which was the gap you were pointing at. That leaves a narrower question. Adopting pre-commit.ci on top would mainly buy auto-fix commits pushed onto contributor branches, and that means granting a third-party app write access, which is a trust decision rather than a coverage one. There is also a concrete blocker either way: the ruff hook is a Closing on the original question, which is now resolved. Happy to revisit the narrower one if you still want it. |
Uh oh!
There was an error while loading. Please reload this page.
Pre-commit.ci integrates changes made making the prioject more stable against contributions not made by the core development team. This does increase both quality control spead and reduced the requirment that each contributer has pre-commit installed locally. This is spececially usefull if you are using an auto formatter such as black or ruff.
All reactions