Skip to content

Drop support for running with 3.4 #6592

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

Merged
merged 4 commits into from
Mar 27, 2019
Merged

Conversation

emmatyping
Copy link
Member

Fixes #6564

This PR was a bit larger than I expected, but after the test failure in #6591 I am keen to move forward with dropping support for 3.4.

Verified

This commit was signed with the committer’s verified signature.
emmatyping Emma Smith
Copy link
Collaborator

@JukkaL JukkaL left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! It's great to be able to drop 3.4 workarounds. I found a few remaining 3.4 references that should probably be updated in tox.ini.

Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, but see Jukka's comment about tox.

I also found references to 3.3 (!) in fastparse.py and fastparse2.py.

Verified

This commit was signed with the committer’s verified signature.
emmatyping Emma Smith

Verified

This commit was signed with the committer’s verified signature.
emmatyping Emma Smith
Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Have you tried to run it with 3.4? Does it even make it to that error message? For all entry points?

@emmatyping
Copy link
Member Author

Have you tried to run it with 3.4? Does it even make it to that error message? For all entry points?

Yes I checked this in a docker container with 3.4.10, it does indeed cause the error to be shown.

I tested mypy,dmypy, and stubgen.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Copy link
Member

@gvanrossum gvanrossum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ship it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants