Skip to content
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

Now that typed_ast 1.3.0 is released, mandate it #6270

Merged
merged 2 commits into from
Jan 30, 2019

Conversation

gvanrossum
Copy link
Member

No description provided.

Copy link
Collaborator

@msullivan msullivan left a comment

Choose a reason for hiding this comment

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

test-requirements.txt needs to be updated too, I think

LG once that is done

@gvanrossum gvanrossum merged commit 871f217 into python:master Jan 30, 2019
@gvanrossum gvanrossum deleted the use-typed-ast-130 branch January 30, 2019 19:53
msullivan pushed a commit that referenced this pull request Jan 31, 2019
(In both setup.py and test-requirements.txt.)
@mr-c
Copy link
Contributor

mr-c commented Feb 4, 2019

Is typed_ast safe or an improvement for mypy 0.660? I noticed some Py3.7 fixes

That is, should I update the typed-ast package in Debian before the final freeze for their upcoming release of Debian Buster? Debian Buster is only shipping Python 3.7 of the Python 3 series..

@gvanrossum
Copy link
Member Author

In general you won't notice much of a difference. But for mypy 0.670 we'll require typed_ast 1.3.x, so you might as well get on that train.

@ethanhs
Copy link
Collaborator

ethanhs commented Feb 4, 2019

Since Buster is only shipping 3.7, and a goal of typed_ast is to be bug-for-bug compatible, I think it would be better to use typed_ast 1.3.x, since the newer typed_ast has better compatibility with (and is based on) the 3.7 ast module. Though if you do, you should probably bump mypy to 0.670 when it is released (hopefully this Friday).

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.

4 participants