-
-
Notifications
You must be signed in to change notification settings - Fork 954
DNM before 2021年09月05日: Remove support for Python 3.5 #1201
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
Conversation
74fe2b8
to
45d1cd5
Compare
Thanks so much! I will merge this one in half a year from now 😁.
ssbarnea
commented
Apr 15, 2021
Why waiting so long for removal of py35? One extra year after py35 already hit EOL seems too much for me.
PS. I say this only because I know the py36 opens the door of static typing, which highly appreciated on any library.
Thanks for sharing the value provided by dropping python 3.5 earlier.
Some types have already been added and for that python 3.4 support was dropped I believe, I assume python 3.6 provides even better ways of inline typing?
In any case, I would be happy to accept a PR which drops python 3.5 support while adding more types in a way that require python 3.6 and thus makes good use of the value added by dropping support earlier.
Just to add (prompted by #1270):
I have been adding comments where typing is impacted by 3.5 support. Searching for "3.5 dropped" gives 9 such comments so far.
Plus the version guard used for os.PathLike (which is just a string in 3.5).
@Yobmod Does that mean this PR needs additional work? Otherwise I would be merging it early, yank the current version on pypi once more and drop python 3.5 support with the next.
Thank you
I dont think any reason to delay this PR or a release.
Once this is merged I'll submit a separate PR with the typing improvements it makes possible.
They won't change much but will cleanup some cast() and type ignores, so can be put in a subsequent release if I don't get to it tonight.
bristea
commented
Jun 15, 2021
Delaying removal of py35 support would do more damage than good to project mainly due to extra maintenance involved.
@Yobmod and @bertwesarg Interestingly now that this branch is merged it fails on CI with failures I can't really fix without spending more time than I have. Maybe it's easy for you to fix due to your experience and I am looking forward to a PR.
In the mean time, I will undo the merge to get main fixed right away. 820d3cc would be the one to revert and take a look at.
Thanks so much!
I reverted the revert...effectively re-adding the removal of python 3.5 as main fails generally now, independently of this commit.
It appears that CI is pulling a more recent version of mypi which now fails where the previous version didn't. Breaking changes seem to be the norm here and we might see this kind of failure more often in the future.
No description provided.