Skip to content

Add third-party six package - #21

Closed
o11c wants to merge 1 commit into
python:masterfrom
o11c:six
Closed

o11c wants to merge 1 commit into
python:masterfrom
o11c:six

Conversation

@o11c

@o11c o11c commented Oct 28, 2015

Copy link
Copy Markdown
Contributor

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

o11c commented Oct 28, 2015

Copy link
Copy Markdown
Contributor Author

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

@o11c

o11c commented Oct 28, 2015

Copy link
Copy Markdown
Contributor Author

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

@o11c

o11c commented Oct 29, 2015

Copy link
Copy Markdown
Contributor Author

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

o11c commented Oct 31, 2015

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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
Copy Markdown
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