Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

Merged
Byron merged 1 commit intogitpython-developers:mainfrombertwesarg:remove-py35
Jun 16, 2021

Conversation

@bertwesarg
Copy link
Contributor

No description provided.

@Byron
Copy link
Member

Thanks so much! I will merge this one in half a year from now 😁.

@ssbarnea
Copy link

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
Copy link
Member

Byron commentedApr 16, 2021
edited
Loading

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
Copy link
Contributor

Yobmod commentedJun 15, 2021
edited
Loading

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).

@Byron
Copy link
Member

@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

@Yobmod
Copy link
Contributor

Yobmod commentedJun 15, 2021
edited
Loading

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.

ssbarnea reacted with thumbs up emoji

@bertwesargbertwesarg marked this pull request as ready for reviewJune 15, 2021 12:55
@bristea
Copy link

Delaying removal of py35 support would do more damage than good to project mainly due to extra maintenance involved.

@ByronByron merged commitcefc0df intogitpython-developers:mainJun 16, 2021
@ByronByron added this to thev3.1.18 - Bugfixes milestoneJun 16, 2021
Byron added a commit that referenced this pull requestJun 16, 2021
This reverts commit45d1cd5.See#1201 which will hopefully help to get a proper fix soon.
@Byron
Copy link
Member

@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!

@Byron
Copy link
Member

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.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@bertwesarg@Byron@ssbarnea@Yobmod@bristea

[8]ページ先頭

©2009-2025 Movatter.jp