Skip to content

Conversation

@rhettinger
Copy link
Contributor

@rhettinger rhettinger commented Aug 6, 2021

@rhettinger rhettinger merged commit 1f7d646 into python:main Aug 6, 2021
@miss-islington
Copy link
Contributor

Thanks @rhettinger for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9, 3.10.
🐍🍒⛏🤖

@bedevere-bot
Copy link

GH-27640 is a backport of this pull request to the 3.10 branch.

@bedevere-bot
Copy link

GH-27641 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 6, 2021
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Aug 6, 2021
@pablogsal
Copy link
Member

@rhettinger Please, make sure you get at least a review from another core Dev before landing changes in the release candidate. I send several communications about this, the last one in this email:

https://mail.python.org/archives/list/[email protected]/thread/RLO5Y6FPOPUJAJVN6MHRIKUSMSRUXUT6/

Here is the important extract:

Once the 3.10 branch reaches RC status, it only can have bug fixes applied that have been reviewed by other core developers (so you cannot merge your own PR without review even if you are a core dev). Generally, these issues must be severe enough (e.g. crashes) that they deserve fixing before the final release. All other issues should be deferred to the next development cycle (Python 3.10.1) since stability is the strongest concern at this point. Also bear in mind that once we reach the RC, the ABI is frozen and cannot change even for bug fixes.
While the goal is to have no code changes between an RC and a final release, there may be a need for final documentation o test fixes. Any such proposed changes should be discussed first with the release manager.
You cannot skip the peer review during an RC, no matter how small! Even if it is a simple copy-and-paste change, everything requires peer review from a core developer

@pablogsal
Copy link
Member

My last message is in reference to the backport, not to this PR.

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

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants