Skip to content

Conversation

verhovsky
Copy link
Contributor

@verhovsky verhovsky commented Dec 30, 2019

Automerge-Triggered-By: @csabella

@csabella
Copy link
Contributor

@verhovsky, thank you for your contribution. I agree with your suggested changes. For the future, this many changes probably could have had a ticket on the bug tracker. Having a ticket raises the visibility of issues, so you have a better chance of a core dev seeing the PR.

Copy link
Contributor

@csabella csabella left a comment

Choose a reason for hiding this comment

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

LGTM

@miss-islington
Copy link
Contributor

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

@miss-islington
Copy link
Contributor

Sorry, @verhovsky, I could not cleanly backport this to 3.8 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 8e1f26e4f06c43cf52afa26ce30f27d2c1129c4a 3.8

@miss-islington miss-islington self-assigned this Dec 31, 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
docs Documentation in the Doc dir skip issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants