Skip to content

Pep 593 py3#639

Merged
ilevkivskyi merged 4 commits into
python:masterfrom
till-varoquaux:pep-593-py3
Jun 5, 2019
Merged

Pep 593 py3#639
ilevkivskyi merged 4 commits into
python:masterfrom
till-varoquaux:pep-593-py3

Conversation

@till-varoquaux
Copy link
Copy Markdown
Contributor

This covers python 3.53 and above. I added a fair amount of tests; this diff is hard to read in this interface because it doesn't seem to offer a way to ignore whitespaces.

Copy link
Copy Markdown
Member

@ilevkivskyi ilevkivskyi left a comment

Choose a reason for hiding this comment

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

Thanks for backporting! Mostly looks good, I just have a bunch of minor comments.

Comment thread typing_extensions/src_py3/test_typing_extensions.py
Comment thread typing_extensions/src_py3/test_typing_extensions.py Outdated
Comment thread typing_extensions/src_py3/test_typing_extensions.py
Comment thread typing_extensions/src_py3/test_typing_extensions.py Outdated
Comment thread typing_extensions/src_py3/test_typing_extensions.py Outdated
Comment thread typing_extensions/src_py3/typing_extensions.py Outdated
Comment thread typing_extensions/src_py3/typing_extensions.py Outdated
Comment thread typing_extensions/src_py3/typing_extensions.py Outdated
Comment thread typing_extensions/src_py3/typing_extensions.py Outdated
Comment thread typing_extensions/src_py3/typing_extensions.py Outdated
@ilevkivskyi
Copy link
Copy Markdown
Member

this diff is hard to read in this interface because it doesn't seem to offer a way to ignore whitespaces.

On the "Files changed tab" there is a little gear icon just above the diff, click on it, and check the box "Ignore whitespace".

Copy link
Copy Markdown
Member

@ilevkivskyi ilevkivskyi left a comment

Choose a reason for hiding this comment

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

Thanks for updates! This is almost ready, I have few more comments.

Comment thread typing_extensions/src_py3/test_typing_extensions.py Outdated
Comment thread typing_extensions/src_py3/test_typing_extensions.py Outdated
raise TypeError("Annotated cannot be used with isinstance().")

def __subclasscheck__(self, cls):
raise TypeError("Annotated cannot be used with issubclass().")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

You marked this as resolved, but nothing has changed here, only in the other version (where it is actually more clear).

Comment thread typing_extensions/src_py3/typing_extensions.py Outdated
Copy link
Copy Markdown
Member

@ilevkivskyi ilevkivskyi left a comment

Choose a reason for hiding this comment

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

Thanks for updates! LGTM now.

@ilevkivskyi ilevkivskyi merged commit 859f57f into python:master Jun 5, 2019
@till-varoquaux till-varoquaux deleted the pep-593-py3 branch June 5, 2019 15:15
jstasiak added a commit to jstasiak/typeshed that referenced this pull request Oct 15, 2019
The code has been added to typing_extensions in
python/typing#632 and
python/typing#639.

I really didn't know how to type Annotated and _AnnotatedAlias here –
feel free to correct this.
JelleZijlstra pushed a commit to python/typeshed that referenced this pull request Oct 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants