Skip to content

Revert __import__ function annotation to return type back to Any#3383

Merged
srittau merged 1 commit intopython:masterfrom
ericvw:revert-__import__-to-return-Any
Oct 17, 2019
Merged

Revert __import__ function annotation to return type back to Any#3383
srittau merged 1 commit intopython:masterfrom
ericvw:revert-__import__-to-return-Any

Conversation

@ericvw
Copy link
Contributor

@ericvw ericvw commented Oct 17, 2019

From python/mypy#7582.

This partially reverts back the change in
0ee7c3c to have __import__ return
Any instead of ModuleType.

Copy link
Collaborator

@srittau srittau left a comment

Choose a reason for hiding this comment

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

Changing the return type back to Any seems sensible, the other changes in the PR were ok, though. A straight revert is not the way to go here.

@ericvw
Copy link
Contributor Author

ericvw commented Oct 17, 2019

Oops — sorry about that. I got a bit overzealous and didn't realize there were other changes desired to keep.

Let me fix that up.

@ericvw ericvw changed the title Revert "Update __import__ function annotations (#3188)" Revert __import__ function annotation to return type back to Any Oct 17, 2019
@ericvw ericvw requested a review from srittau October 17, 2019 21:11
From python/mypy#7582.

This partially reverts back the change in
0ee7c3c to have `__import__` return
`Any` instead of `ModuleType`.
@ericvw
Copy link
Contributor Author

ericvw commented Oct 17, 2019

@srittau, thanks for pointing that out.

The PR should be just addressing changing the return type back to Any now.

Copy link
Collaborator

@srittau srittau left a comment

Choose a reason for hiding this comment

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

Thanks!

@srittau srittau merged commit 966f8d2 into python:master Oct 17, 2019
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