Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork966
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Byron commentedMar 13, 2021
Thanks so much! I will merge this one in half a year from now 😁. |
ssbarnea commentedApr 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. |
Byron commentedApr 16, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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. |
Yobmod commentedJun 15, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Just to add (prompted by#1270): |
Byron commentedJun 15, 2021
@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. |
Yobmod commentedJun 15, 2021 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
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 commentedJun 15, 2021
Delaying removal of py35 support would do more damage than good to project mainly due to extra maintenance involved. |
Byron commentedJun 16, 2021
@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. |
Byron commentedJun 16, 2021
I reverted the revert…effectively re-adding the removal of python 3.5 as main fails generally now, independently of this commit. |
No description provided.