-
Notifications
You must be signed in to change notification settings - Fork 195
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
Update black version #1010
Update black version #1010
Conversation
Thanks Vyas! 😄 Looks like we are getting some failures from |
Yup, the version of black in the build environment won't actually be updated until the PR that you linked gets merged. Then the style checks here should pass. |
rerun tests |
This is a backwards-compatible alternative to #1010 that does not require modifying linter behavior as we are in the middle of code freeze. Authors: - Vyas Ramasubramani (https://github.com/vyasr) Approvers: - https://github.com/jakirkham - GALI PREM SAGAR (https://github.com/galipremsagar) - Jordan Jacobelli (https://github.com/Ethyling)
Is this pulling the right version of
|
rerun tests |
This file needs to be updated as well: https://github.com/rapidsai/rmm/blob/branch-22.06/python/dev_requirements.txt (Or maybe we just need to remove it? The conda environment has similar specifications. Also the dev requirements can be met entirely with pip/scikit-build + pre-commit now, I think, so the extra file may not be needed.) |
This should just be using whatever's in the integration repo, which is why tests were failing yesterday. However, now images are out and tests are still failing so I'm investigating further.
I'm not sure what to do with this. I believe it was originally added to support a Dockerfile that is no longer maintained within the repo. I'll just update it for now and we can address the potential cleanup in a subsequent PR. |
I've pulled the image locally and for some reason it's still got the wrong (old) version of black. My best guess is that the mixing of rapidsai/integration#452 and rapidsai/integration#455 caused the integration repo to upload slightly out-of-date images. I'll follow up with ops. |
It looks like the built images were indeed based on slightly out of date files. Once new versions are up we can run tests again. |
It seems the |
# Conflicts: # python/dev_requirements.txt
The Docker images haven't been updated yet from the ones we had last night, which were the out-of-date ones. @ajschmidt8 triggered new builds but they don't appear to have been uploaded yet. |
rerun tests |
1 similar comment
rerun tests |
So CI is pulling in the image that was built last night, but we are still seeing a failure here |
rerun tests |
Hooray the style check passes! 🎉 |
Is there anything else that needs to be done here or are we safe to merge? |
@gpucibot merge |
This PR updates to black 22.3.0 which contains a critical bugfix for a recent release of Click 8.1.0. It also includes some minor reformatting associated with the version bump.