Skip to content

bpo-39019: Implement missing __class_getitem__ for SpooledTemporaryFile#17560

Merged
ilevkivskyi merged 5 commits into
python:masterfrom
isidentical:bpo-39019-2
Dec 30, 2019
Merged

bpo-39019: Implement missing __class_getitem__ for SpooledTemporaryFile#17560
ilevkivskyi merged 5 commits into
python:masterfrom
isidentical:bpo-39019-2

Conversation

@isidentical
Copy link
Copy Markdown
Member

@isidentical isidentical commented Dec 10, 2019

Comment thread Lib/test/test_tempfile.py Outdated
Comment thread Misc/NEWS.d/next/Library/2019-12-10-21-11-05.bpo-39019.YIlgZ7.rst 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! Here are couple comments.

Comment thread Lib/tempfile.py
Comment thread Misc/NEWS.d/next/Library/2019-12-10-21-11-05.bpo-39019.YIlgZ7.rst Outdated
Comment thread Lib/tempfile.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.

LGTM!

@ilevkivskyi ilevkivskyi merged commit 09c482f into python:master Dec 30, 2019
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Jan 31, 2020
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.

5 participants