Skip to content

bpo-46831: Update __build_class__ comment #31522

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Mar 3, 2022

Conversation

hauntsaninja
Copy link
Contributor

@hauntsaninja hauntsaninja commented Feb 23, 2022

@hauntsaninja
Copy link
Contributor Author

hauntsaninja commented Feb 23, 2022

I got a little bit confused by this recently. This was changed in Python 3.4 in e8e1459 and a lot of __build_class__ examples on the internet are outdated.

@JelleZijlstra JelleZijlstra self-assigned this Feb 25, 2022
Co-authored-by: Jelle Zijlstra <[email protected]>
Copy link
Member

@JelleZijlstra JelleZijlstra left a comment

Choose a reason for hiding this comment

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

@gvanrossum planning to merge this change (fixing a confusing comment in C code for __build_class__)

@JelleZijlstra JelleZijlstra added needs backport to 3.9 only security fixes needs backport to 3.10 only security fixes labels Mar 3, 2022
@JelleZijlstra JelleZijlstra merged commit 81d968b into python:main Mar 3, 2022
@miss-islington
Copy link
Contributor

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

@bedevere-bot
Copy link

@JelleZijlstra: Please replace # with GH- in the commit message next time. Thanks!

@bedevere-bot
Copy link

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

@bedevere-bot bedevere-bot removed the needs backport to 3.10 only security fixes label Mar 3, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 3, 2022
Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit 81d968b)

Co-authored-by: Shantanu <[email protected]>
@bedevere-bot
Copy link

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

@bedevere-bot bedevere-bot removed the needs backport to 3.9 only security fixes label Mar 3, 2022
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Mar 3, 2022
Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit 81d968b)

Co-authored-by: Shantanu <[email protected]>
miss-islington added a commit that referenced this pull request Mar 3, 2022
Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit 81d968b)

Co-authored-by: Shantanu <[email protected]>
miss-islington added a commit that referenced this pull request Mar 3, 2022
Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit 81d968b)

Co-authored-by: Shantanu <[email protected]>
@hauntsaninja hauntsaninja deleted the buildcl branch March 3, 2022 06:27
hello-adam pushed a commit to hello-adam/cpython that referenced this pull request Jun 2, 2022
Co-authored-by: Jelle Zijlstra <[email protected]>
(cherry picked from commit 81d968b)

Co-authored-by: Shantanu <[email protected]>
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