-
Notifications
You must be signed in to change notification settings - Fork 131
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add support for pre-commit-ci #476
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #476 +/- ##
=======================================
Coverage 90.82% 90.82%
=======================================
Files 21 21
Lines 1972 1972
=======================================
Hits 1791 1791
Misses 181 181 ☔ View full report in Codecov by Sentry. |
ruff has a v2 logo that is in their repo. Shown here |
This PR was actually a no-op. It creates a config file with all the default settings. It would be useful for customizing the configuration in the future, but it could also be deleted. I think the pre-commit docs suck (I've attempted PRs to improve them but was rejected). I'll explain the steps here for enabling pre-commit-ci for this repo.
After entering your authentication code you should be all set. I just did the procedure for this repo, so everything should be working now! |
Thank you very much @maresb 🤜🤛 |
Motivated by #473
I suggest adding support for pre-commit-ci to automate the revision updates and potentially improve the dev experience by auto-fixing PRs. See docs: https://pre-commit.ci/. I added the defaults explicitly to have more control over the actions.
Type of change
📚 Documentation preview 📚: https://pymc-marketing--476.org.readthedocs.build/en/476/