Skip to content

Conversation

rchen152
Copy link
Collaborator

@rchen152 rchen152 commented Oct 8, 2020

I'm trying to pull the latest version of typeshed into Google, and
pytype chokes on pstats in Python 2 because the enum module was
introduced in 3.4.

I'm trying to pull the latest version of typeshed into Google, and
pytype chokes on pstats in Python 2 because the enum module was
introduced in 3.4.
Copy link
Collaborator

@hauntsaninja hauntsaninja left a comment

Choose a reason for hiding this comment

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

We could move it under the branch that defines SortKey.

I assume typeshed CI didn't complain about this for Python 2 because it looks at third_party/2/enum.pyi.

@rchen152
Copy link
Collaborator Author

rchen152 commented Oct 8, 2020

Putting the import in the SortKey branch does look nicer. Done.

We don't import most of the third_party directory, so yeah, that's likely the difference.

@rchen152 rchen152 merged commit fff7a10 into master Oct 8, 2020
@rchen152 rchen152 deleted the pstats branch October 8, 2020 23:09
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.

2 participants