Skip to content

drop support for python 3.6 - #4720

Merged
jhamman merged 17 commits into
pydata:masterfrom
keewis:drop-py36
Jan 7, 2021
Merged

drop support for python 3.6#4720
jhamman merged 17 commits into
pydata:masterfrom
keewis:drop-py36

Conversation

@keewis

@keewiskeewis commented Dec 21, 2020•
edited
Loading

Copy link
Copy Markdown
Collaborator

Following NEP-29, our dependencies have started to drop active support for python 3.6. I tried removing all mentions of py36, but I might have missed something.

Last time I checked support for python 3.9 was blocked by some dependencies, so adding that will have to wait a bit longer.

@keewis

keewis commented Dec 21, 2020

Copy link
Copy Markdown
CollaboratorAuthor

it seems we have to update the minimum setuptools version for this, but I'm not sure which to choose. 39.2 is the last release before py37 was released, but I can't get conda (using conda-forge) to install anything below 40.4 into a environment with python=3.7.9

@mathause

mathause commented Dec 22, 2020

Copy link
Copy Markdown
Collaborator

https://github.com/pydata/xarray/blob/master/doc/installing.rst will need to be adjusted

keewis
keewis commented Dec 22, 2020
View reviewed changes
Comment threaddoc/installing.rst Outdated
@keewis

keewis commented Dec 22, 2020

Copy link
Copy Markdown
CollaboratorAuthor

I will also change the whats-new.rst entry. @mathause, was there anything missing other than the setuptools version?

@mathause

mathause commented Dec 22, 2020

Copy link
Copy Markdown
Collaborator

I think thats all.

@jhamman

jhamman commented Dec 23, 2020

Copy link
Copy Markdown
Member

LGTM. As I understand it, we don't really have a choice on the setuptools version bump. Can we temporarily disable the check on setuptools so the CI passes?

@keewis

keewis commented Dec 26, 2020

Copy link
Copy Markdown
CollaboratorAuthor

there's a POLICY_OVERRIDE for setuptools, so I updated that.

Reading #4295 again, I'm not sure what to do about this: we don't really need to require setuptools>=40.4 because of features / bug fixes, it's just that we can't have our CI check that that version doesn't break anything (I don't expect using the pip section works). I think I will update the changelog entry to reflect that, and not update the install_requires / setup_requires entries for setuptools.

We will only be able to fully get rid of this issue once we drop support for python 3.7 and are able to switch to the stdlib importlib.* libraries (and the external entrypoints library).

This was referenced Dec 30, 2020
Merged
Merged
@keewiskeewis closed this Jan 6, 2021
@keewiskeewis reopened this Jan 6, 2021
@mathause

mathause commented Jan 6, 2021

Copy link
Copy Markdown
Collaborator

Do you need to update setuptools in setup.cfg?

xarray/setup.cfg

Line 79 in 31d540f

setuptools >= 38.4 # For pkg_resources

@keewis

keewis commented Jan 6, 2021

Copy link
Copy Markdown
CollaboratorAuthor

Do you need to update setuptools in setup.cfg?

👍 we should probably keep that in sync with the environment files.

@shoyer, any opinion about the bump of setuptools beyond the maximum version available when py37 was released (see #4720 (comment))?

Otherwise this should be ready for merging.

@jhamman

jhamman commented Jan 6, 2021

Copy link
Copy Markdown
Member

My opinion on the bump of setuptools is that we don't really have a choice. If 40.4 is the minimum version that works, that's the minimum version that works. We should really just be treating the minimum version policy as a guide in cases like this. In this case, we make an exception because of incompatibility outside our control.

@jhamman

jhamman commented Jan 7, 2021

Copy link
Copy Markdown
Member

This is blocking #4730. Any objections to merging this today?

@jhamman
jhamman merged commit 477e75a into pydata:masterJan 7, 2021
@keewis
keewis deleted the drop-py36 branch January 7, 2021 18:39
@raybellwavesraybellwaves mentioned this pull request Jan 7, 2021
Closed
dcherian added a commit to TomNicholas/xarray that referenced this pull request Jan 18, 2021
@dcherian
Merge remote-tracking branch 'upstream/master' into integrate_dim_arg
b9c1e6f
* upstream/master: (342 commits)
fix decode for scale/ offset list (pydata#4802)
Expand user dir paths (~) in open_mfdataset and to_zarr. (pydata#4795)
add a version info step to the upstream-dev CI (pydata#4815)
fix the ci trigger action (pydata#4805)
scatter plot by order of the first appearance of hue (pydata#4723)
don't skip the scheduled CI (pydata#4806)
coords: retain str dtype (pydata#4759)
Fix interval labels with units (pydata#4794)
Always force dask arrays to float in missing.interp_func (pydata#4771)
Print number of variables in repr (pydata#4762)
install conda as a library in the minimum dependency check CI (pydata#4792)
Migrate CI from azure pipelines to GitHub Actions (pydata#4730)
use conda.api instead of parallel calls to the conda binary (pydata#4775)
Speed up missing._get_interpolator (pydata#4776)
Remove special case in guess_engines (pydata#4777)
improve typing of OrderedSet (pydata#4774)
CI: ignore some warnings (pydata#4773)
DOC: update hyperlink for xskillscore (pydata#4778)
drop support for python 3.6 (pydata#4720)
Trigger upstream CI on cron schedule (by default) (pydata#4729)
...
@aulemahalaulemahal mentioned this pull request Jan 18, 2021
Closed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jhammanjhammanjhamman approved these changes
@dcheriandcheriandcherian approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

drop python 3.6 support

4 participants

@keewis@mathause@jhamman@dcherian