-
Notifications
You must be signed in to change notification settings - Fork 35
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 #296
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
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 good pending branch update
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.
Thanks for adding a test build.
I need to wait until the new secrets are added and then this can be merged. |
Problem
We moved all linting and formatting dependencies into
.pre-commit-config.yml
. While this consolidates these tools and isolates the dev environment from these dependencies, it also hides them fromdependabot
.Solution
Add a reusable workflow in
dbt-labs/actions
and call it here. This workflow will:pre-commit autoupdate
pre-commit run --all-files
CI will take care of the verification of whether new conflicts were found in the updated
pre-commit
hooks.The resulting PR: #297
Checklist