-
Notifications
You must be signed in to change notification settings - Fork 2
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
Revert "[pre-commit.ci] pre-commit autoupdate" #482
Conversation
This reverts commit 8c49b50.
Another issue is this isn't going to update the template's ones. We don't want this on at all. |
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.
I think it's good to keep these up to date, so see no reason to revert this. If you want to disable pre-commit.ci auto updating the versions, please could you open a new issue and we can discuss there?
@renovatebot will do it in 3 days... |
🤷 I'm ok with alphas. But if we decide never to take pre-commit autoupdates and only use @renovate: can we actually disable it in the |
I've re-opened #463 to discuss, so we can make sure discussion is in one place for future reference |
It's not possible pre-commit-ci/issues#83 |
Also the problem is this doesn't update the template which @renovatebot would |
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.
Let's revert this and add the bot back?
Testing the bot with forbidden branch naming rule
Reverts #479
We don't want the alpha version