-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CI: pre-commit autoupdate to fix CI #46558
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
Conversation
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 @MarcoGorelli lgtm
OK to merge once Code Checks / pre-commit is green (locally OK so should be no different here)
I assume will will get the same error on 1.4.x? |
hmm, merged #46550 a couple of hours ago after the problem was noticed on main. pre-commit was green on 1.4.x. more investigation required. |
I think it's just that on the 1.4.x branch the run used the old cached dependencies:
which would've had the previous Don't know how long the cache lasts, but I think this should probably be backported |
merging then |
This comment was marked as resolved.
This comment was marked as resolved.
@meeseeksdev backport 1.4.x |
This comment was marked as resolved.
This comment was marked as resolved.
* pre-commit autoupdate * update environment.yml
new click release broke previous version of
black
, see psf/black#2964