-
Notifications
You must be signed in to change notification settings - Fork 219
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
Unify pre-commit config across repos #330
Conversation
Integration of the frontend with this repository is pending BetaHuhn/repo-file-sync-action#271. We could theoretically use the action from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Really nice; lgtm.
* Use retry library to catch and retry SSLErrors * Add test for retry condition * Pin retry version
Fixes
Fixes #316 by @dhruvkb
Description
This PR improves the pre-commit sync by importing bits from the API and the catalog and then configures the sync action to push the changes downstream.
It is accompanied by two other PRs which should be merged before this one.
Checklist
Update index.md
).main
) or a parent feature branch.Developer Certificate of Origin
Developer Certificate of Origin