Conversation
32ae2c1 to
0d0e88c
Compare
|
I can squash this to one commit if we want. |
|
(prefer this being squashed) |
|
I left each minimum version change as its own commit but squashed the CI thrashing. |
| EXTRAREQS: "-r requirements/testing/travis_extra.txt" | ||
| - PYTHON_VERSION: "3.7" | ||
| - PYTHON_VERSION: "3.8" | ||
| CONDA_INSTALL_LOCN: "C:\\Miniconda37-x64" |
There was a problem hiding this comment.
| CONDA_INSTALL_LOCN: "C:\\Miniconda37-x64" | |
| CONDA_INSTALL_LOCN: "C:\\Miniconda38-x64" |
There was a problem hiding this comment.
This doesn't work unless we update the windows image we are using.
| # For building, use a new environment | ||
| - conda create -q -n test-environment python=%PYTHON_VERSION% tk | ||
| - activate test-environment | ||
| # pull pywin32 from conda because on py38 there is something wrong with finding |
There was a problem hiding this comment.
Is that because of wrong conda above?
There was a problem hiding this comment.
I don't think so, we are still ending up with a py38 env and it should be de-coupled from Python version of the "base" conda enivorment. I think this has to do with Python tightening up its rules about how dlls are discovered. There are a bunch of related issues on the pywin32 issues tracker.
a8f1e20 to
924e1cf
Compare
|
LGTM, but waiting on the API change note. |
This was release in March 2018 and is the first version of python-dateutil that does not have deprecation warnings due to the collections -> collections.abc change in CPython.
This is released in Sept 2018 and is the earliest version to not warn due to the collections -> collections.abc change in CPython.
This is consistent with our policy and NEP29
PR Summary
Drop support for py36
closes #17652
PR Checklist
This should not go in until after we branch v3.3.x off of master and still needs an API change note added (but I want to let the dust settle on the renaming / moving associated with 3.3 before doing that.