◐ Shell
reader mode source ↗
Skip to content

bpo-36889: Replace deprecation warning with RuntimeError#14397

Merged
asvetlov merged 1 commit into
python:masterfrom
asvetlov:stricter-check
Jun 27, 2019
Merged

bpo-36889: Replace deprecation warning with RuntimeError#14397
asvetlov merged 1 commit into
python:masterfrom
asvetlov:stricter-check

Conversation

@asvetlov

@asvetlov asvetlov commented Jun 26, 2019

Copy link
Copy Markdown
Contributor

asyncio.Stream is a new class, there is no reason for deprecation period

https://bugs.python.org/issue36889

@eamanu eamanu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hide comment

LGTM

@tirkarthi tirkarthi left a comment

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.

Hide comment

LGTM. Thanks.

@asvetlov asvetlov merged commit 97d15b1 into python:master Jun 27, 2019
@miss-islington

Copy link
Copy Markdown
Contributor

Thanks @asvetlov for the PR 🌮🎉.. I'm working now to backport this PR to: 3.8.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 27, 2019
(cherry picked from commit 97d15b1)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
@bedevere-bot

Copy link
Copy Markdown

GH-14422 is a backport of this pull request to the 3.8 branch.

@asvetlov asvetlov deleted the stricter-check branch June 27, 2019 11:39
miss-islington added a commit that referenced this pull request Jun 27, 2019
(cherry picked from commit 97d15b1)

Co-authored-by: Andrew Svetlov <andrew.svetlov@gmail.com>
lisroach pushed a commit to lisroach/cpython that referenced this pull request Sep 10, 2019
DinoV pushed a commit to DinoV/cpython that referenced this pull request Jan 14, 2020
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.

6 participants