-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
black fails due to _unicodefun #2963
Labels
C: packaging
Installation and packaging of Black
R: duplicate
This issue or pull request already exists
T: bug
Something isn't working
Comments
I found a solution: this is a issue with the newest |
Yep. Click 8.1.0 was released with a breaking change. |
Going to close in favor of #2964 which is now pinned. |
ichard26
added
R: duplicate
This issue or pull request already exists
C: packaging
Installation and packaging of Black
labels
Mar 28, 2022
thejcannon
added a commit
to pex-tool/pex
that referenced
this issue
Mar 28, 2022
thejcannon
added a commit
to pex-tool/pex
that referenced
this issue
Mar 28, 2022
This was referenced Apr 1, 2022
rapids-bot bot
pushed a commit
to rapidsai/dask-cuda
that referenced
this issue
Apr 4, 2022
Looks like the version of Black we were using broke due to a change in click (xref psf/black#2963) - this PR bumps us to the latest version, which resolves those issues (and makes some minor styling changes) Authors: - Charles Blackmon-Luca (https://github.com/charlesbluca) Approvers: - GALI PREM SAGAR (https://github.com/galipremsagar) - Peter Andreas Entschev (https://github.com/pentschev) URL: #889
rapids-bot bot
pushed a commit
to rapidsai/cuspatial
that referenced
this issue
Apr 6, 2022
…rect column indexing (#511) This PR resolves three separate issues caused by changes to our dependencies: 1. The version of Black we were using broke due to a change in click (xref psf/black#2963). This PR bumps us to the latest version, which resolves those issues (and makes some minor styling changes). 2. rapidsa/cudf#10589 changed the `CUDA_TRY` macro to `CUDF_CUDA_TRY` and `CHECK_CUDA` to `CUDF_CHECK_CUDA`, so cuspatial needs to make that change as well. 3. rapids/cudf#10516 removed support for direct indexing of columns via `__getitem__`. This PR replaces those calls with the appropriate methods. Authors: - Charles Blackmon-Luca (https://github.com/charlesbluca) - Vyas Ramasubramani (https://github.com/vyasr) Approvers: - Paul Taylor (https://github.com/trxcllnt) - Mark Harris (https://github.com/harrism) URL: #511
pajowu
added a commit
to okfde/froide
that referenced
this issue
Apr 7, 2022
22.3 contains the fix for psf/black#2963
stefanw
pushed a commit
to okfde/froide
that referenced
this issue
Apr 7, 2022
22.3 contains the fix for psf/black#2963
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
C: packaging
Installation and packaging of Black
R: duplicate
This issue or pull request already exists
T: bug
Something isn't working
Describe the bug
On Centos 7:
Additional context
This just popped up today in CI; it was not observed earlier. Maybe the dependencies are not tightly pinned?
The text was updated successfully, but these errors were encountered: