-
Notifications
You must be signed in to change notification settings - Fork 7
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 pre-commit autoupdate workflow #212
Conversation
TestingI tested this workflow on my fork and it worked as expected when set to run when the main branch was pushed to. I haven't tested the weekly schedule, since that requires waiting until it runs, so that will have to wait. |
@xylar Can you tell me if you see anything obvious that should be changed about this workflow? |
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.
@altheaden, do Push Changes
and Make PR
not get called if there are no changes? Or do they fail? If the latter, that might be confusing especially since there's no text to say that there were no changes.
Very nice work, by the way! I'm looking forward to seeing this in action! |
fb12c71
to
98387c6
Compare
98387c6
to
203d432
Compare
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.
Looks superb!
This adds a GitHub actions workflow that will run the
pre-commit autoupdate
command to update pre-commit and its dependencies. The workflow is set to run weekly on Wednesdays.Checklist
Testing
comment in the PR documents testing used to verify the changes