◐ Shell
clean mode source ↗

bpo-36404: update PendingDeprecationWarning explanation by methane · Pull Request #12837 · python/cpython

Skip to content

Navigation Menu

Sign in

Appearance settings

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up

Appearance settings

Conversation

@methane

@methane methane commented

Apr 15, 2019

edited by bedevere-bot

Loading

Copy link Copy Markdown

Member

@ncoghlan ncoghlan merged commit a3283ef into python:master

Apr 15, 2019

@miss-islington

Copy link Copy Markdown

Contributor

Thanks @methane for the PR, and @ncoghlan for merging it 🌮🎉.. I'm working now to backport this PR to: 3.7.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request

Apr 15, 2019
Keep the nudge towards DeprecationWarning, but remove the
"Note" markup and generally shorten the description.

Ref: https://github.com/python/cpython/pull/12505/filesGH-r273978757
(cherry picked from commit a3283ef)

Co-authored-by: Inada Naoki <songofacandy@gmail.com>

@bedevere-bot

Copy link Copy Markdown

GH-12838 is a backport of this pull request to the 3.7 branch.

@ncoghlan

Copy link Copy Markdown

Contributor

Thanks @methane!

miss-islington added a commit that referenced this pull request

Apr 15, 2019
Keep the nudge towards DeprecationWarning, but remove the
"Note" markup and generally shorten the description.

Ref: https://github.com/python/cpython/pull/12505/filesGH-r273978757
(cherry picked from commit a3283ef)

Co-authored-by: Inada Naoki <songofacandy@gmail.com>

@methane methane deleted the pendingdeprecatedwarning branch

May 29, 2019 10:27

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ncoghlan ncoghlan ncoghlan approved these changes

Assignees

No one assigned

Labels

docs

Documentation in the Doc dir

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@methane @miss-islington @bedevere-bot @ncoghlan @the-knights-who-say-ni