-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Make sure setuptools is compatible with PEP 639 #10512
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
base: main
Are you sure you want to change the base?
Conversation
I think this error is saying that the thing in
or
which is a link to the file itself |
Right, requiring However, if I understand you correctly, this is not compatible with Numpy 2.2: xarray/asv_bench/asv.conf.json Line 63 in 22bd964
We'll probably have to partially revert #10300 and fall back to pre-PEP639 format. Wait, no, according to #10355 there's more to that. I guess the real question is why I saw this issue surface recently in CI tests? |
8ac6a51
to
d0c8f33
Compare
d0c8f33
to
8a211f3
Compare
PEP 639 requires setuptools ≥ 77. It seems customary to require setuptools ≥ 77.0.2 which irons out some bugs related to PEP 639 support.
281ea27
to
7defc56
Compare
7defc56
to
ba5c608
Compare
The error you mention above is from the asv benchmarks (only running CI after adding the corresponding label), for which Xarray is installed with |
Instead rely onproject.license
inpyproject.toml
.Let's see if this fixes this CI error:
whats-new.rst
api.rst