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

Add third-party six package #21

Closed
wants to merge 1 commit into from
Closed

Add third-party six package #21

wants to merge 1 commit into from

Conversation

o11c
Copy link
Contributor

@o11c o11c commented Oct 28, 2015

Things of note:

  • While I believe types is now complete, it still needs special handling in checkers. Some of this might already be possible with typing, but some of it is certainly not.
  • Some of the stubs in six.moves use the .py extension so they can use glob exports, since sys.modules trickery is out of scope for tools that use stubs.
  • I ignore the difference between imp.reload and importlib.reload in python 3.4+, the signature is the same.
  • Many stubs can be written in 2and3 now (this was actually my motivation).

@o11c
Copy link
Contributor Author

o11c commented Oct 28, 2015

Whoops, forgot to run the checks on the final version, will fix and rebase in a bit.

@o11c
Copy link
Contributor Author

o11c commented Oct 28, 2015

Okay, done now, except that some of the modules needed by six.moves aren't stubbed yet.

@o11c
Copy link
Contributor Author

o11c commented Oct 29, 2015

It looks like all the review comments are for the six.moves part. Since #22 isn't done either, I think I'll split that out of this PR.

@o11c
Copy link
Contributor Author

o11c commented Oct 31, 2015

I believe I've addressed all the review comments now, except that if this is merged now a lot of tests will fail, until #22 is finished.

@matthiaskramm
Copy link
Contributor

Yep, this still makes a lot of mypy tests fail.

timabbott added a commit to timabbott/typeshed that referenced this pull request Jan 26, 2016
timabbott added a commit to timabbott/typeshed that referenced this pull request Jan 26, 2016
timabbott added a commit to timabbott/typeshed that referenced this pull request Jan 26, 2016
timabbott added a commit to timabbott/typeshed that referenced this pull request Jan 27, 2016
timabbott added a commit to timabbott/typeshed that referenced this pull request Jan 27, 2016
@gvanrossum gvanrossum mentioned this pull request Jan 27, 2016
@gvanrossum gvanrossum changed the title Add third-party six package and finish types Add third-party six package Jan 27, 2016
@gvanrossum
Copy link
Member

Closing this since it's out of date; however I'm keeping a pointer in #66 just in case.

@gvanrossum gvanrossum closed this Jan 27, 2016
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.

3 participants