-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
DEPS: update setuptools min version #42304
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
DEPS: update setuptools min version #42304
Conversation
are we not testing with the minimum setuptools? |
No, we are not. |
in doc/source/development/contributing_environment.rst
can be updated. |
can you update the min pin in environment.yaml (and other places if needed)? |
also this should be added to the wheel recipe @simonjayhawkins (practically this already works, but should add) |
test failure unrelated
|
@fangchenli can you address this |
pandas-feedstock or MacPython or both? |
We don't list setuptools in |
labelled as blocker as we won't be changing min versions of dependencies in patch releases? |
yeah i think we should just add this to |
conda-incubator/setup-miniconda@v2 does not seem to like the setuptools pin, although I think there have been conda-incubator/setup-miniconda@v2 timeouts on other PRs |
The version in pandas-dev on CI / Checks for master (https://github.com/pandas-dev/pandas/runs/2950514145) is
and creating dev env locally my backport env (1.3.x)
my master env
hmm? our tests pass locally? |
@jreback ok to release without this PR, or delay release to allow time for others to look at this? @pandas-dev/pandas-core |
should |
it appears can't create a dev env with conda with this pin |
Could leave it out of the yml file and then do a pip install -r requirements-dev.txt (assuming it is in requirements-dev.txt). This will no op everything that is installed, but upgrade setuptools if below. |
can create a dev env locally from scratch using this pin. so maybe something related to conda-incubator/setup-miniconda@v2 creating a dev env from scratch without this pin, the only diff is
so it's a mystery to me how conda chooses the version. |
hmm yeah this should just work. |
moving to 1.3.2 |
This comment has been minimized.
This comment has been minimized.
thanks @fangchenli this is essentially a doc update at this point so backporting ok. |
@meeseeksdev backport 1.3.x |
This comment has been minimized.
This comment has been minimized.
Co-authored-by: Fangchen Li <[email protected]>
According to https://setuptools.readthedocs.io/en/latest/userguide/declarative_config.html, the min version of setuptools that support
entry_points
insetup.cfg
is51.0.0
.