Skip to content

Fix "Py_TPFLAGS_MANAGED_WEAKREF is set in tp_flags"#112237

Merged
kumaraditya303 merged 2 commits into
python:mainfrom
da-woods:patch-1
Jul 3, 2024
Merged

Fix "Py_TPFLAGS_MANAGED_WEAKREF is set in tp_flags"#112237
kumaraditya303 merged 2 commits into
python:mainfrom
da-woods:patch-1

Conversation

@da-woods

@da-woods da-woods commented Nov 18, 2023

Copy link
Copy Markdown
Contributor

Fix a small error in the documentation: Py_TPFLAGS_MANAGED_WEAKREF is set in the tp_flags field rather than the tp_dict field


📚 Documentation preview 📚: https://cpython-previews--112237.org.readthedocs.build/

Fix a small error in the documentation: `Py_TPFLAGS_MANAGED_WEAKREF` is set in the `tp_flags` field rather than the `tp_dict` field
@bedevere-app bedevere-app Bot added awaiting review docs Documentation in the Doc dir skip news labels Nov 18, 2023

@Eclips4 Eclips4 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.

Thanks!

@da-woods

Copy link
Copy Markdown
Contributor Author

@Eclips4 Sorry for bumping it after you approved it. I spotted a second error in the same section that seemed worth fixing at the same time.

Comment thread Doc/c-api/typeobj.rst
@arhadthedev arhadthedev added needs backport to 3.11 only security fixes needs backport to 3.12 only security fixes labels Nov 18, 2023
@da-woods

Copy link
Copy Markdown
Contributor Author

@markshannon Pinging you because I think both changes in this PR touch lines that you added. This is a fairly trivial documentation fix I think.

@Eclips4 Eclips4 requested a review from markshannon January 26, 2024 10:56
@hugovk hugovk removed the needs backport to 3.11 only security fixes label Apr 11, 2024
@serhiy-storchaka serhiy-storchaka added the needs backport to 3.13 bugs and security fixes label May 9, 2024
@miss-islington-app

Copy link
Copy Markdown

Thanks @da-woods for the PR, and @kumaraditya303 for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 3, 2024
…12237)

(cherry picked from commit 4232976)

Co-authored-by: da-woods <dw-git@d-woods.co.uk>
@bedevere-app

bedevere-app Bot commented Jul 3, 2024

Copy link
Copy Markdown

GH-121309 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Jul 3, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jul 3, 2024
…12237)

(cherry picked from commit 4232976)

Co-authored-by: da-woods <dw-git@d-woods.co.uk>
@bedevere-app

bedevere-app Bot commented Jul 3, 2024

Copy link
Copy Markdown

GH-121310 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.12 only security fixes label Jul 3, 2024
kumaraditya303 pushed a commit that referenced this pull request Jul 3, 2024
…112237) (#121310)

docs: Fix "Py_TPFLAGS_MANAGED_WEAKREF is set in tp_flags" (GH-112237)
(cherry picked from commit 4232976)

Co-authored-by: da-woods <dw-git@d-woods.co.uk>
kumaraditya303 pushed a commit that referenced this pull request Jul 3, 2024
…112237) (#121309)

docs: Fix "Py_TPFLAGS_MANAGED_WEAKREF is set in tp_flags" (GH-112237)
(cherry picked from commit 4232976)

Co-authored-by: da-woods <dw-git@d-woods.co.uk>
@da-woods da-woods deleted the patch-1 branch July 3, 2024 10:05
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
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.

6 participants