-
Notifications
You must be signed in to change notification settings - Fork 111
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
Upgrade black for new remote-dev boxes #2900
Conversation
@csjiang in order for this PR to pass, we need to rebuild remote-dev with the newest version of black. New remote-dev boxes won't be able to run pre-commit hooks until this PR is in. If other dev environments do not auto-upgrade black, then they need to run:
|
@joaquincasares gotcha on the remote dev stuff but i think to get PR to pass CI checks is just fixing the spacing w the operator on those two extra files right? i'll rebake remote dev after this goes into master to get ahead of the pre-commit stuff, but can we fix those two files so this PR can be merged? |
Sounds good! I'll ping you when this is merged and then open up a second PR. Thanks! |
Description
We are seeing this with new remote-dev boxes:
-- pallets/click#2225
Tests
How will this change be monitored? Are there sufficient logs?