Skip to content
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

gh-127443: fix some entries in Doc/data/refcounts.dat #127451

Merged
merged 2 commits into from
Dec 2, 2024

Conversation

picnixz
Copy link
Contributor

@picnixz picnixz commented Nov 30, 2024

This fixes some entries that were detected by my linter (still in progress). I suspect more entries will be fixed when the tool is ready, but I think we can already fix those.


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

Copy link
Member

@Eclips4 Eclips4 left a comment

Choose a reason for hiding this comment

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

Thank you, LGTM. FYI, I plan to merge it tomorrow.

@picnixz
Copy link
Contributor Author

picnixz commented Dec 1, 2024

@Eclips4 I fixed some additional entries that my tool didn't find (I hadn't implemented the check yesterday)

Copy link
Member

@ZeroIntensity ZeroIntensity left a comment

Choose a reason for hiding this comment

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

Re-approving

@Eclips4 Eclips4 merged commit 1f8267b into python:main Dec 2, 2024
25 checks passed
@miss-islington-app
Copy link

Thanks @picnixz for the PR, and @Eclips4 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 Dec 2, 2024
…GH-127451)

Fix incorrect entries in `Doc/data/refcounts.dat`
(cherry picked from commit 1f8267b)

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 2, 2024
…GH-127451)

Fix incorrect entries in `Doc/data/refcounts.dat`
(cherry picked from commit 1f8267b)

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
@bedevere-app
Copy link

bedevere-app bot commented Dec 2, 2024

GH-127496 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 Dec 2, 2024
@bedevere-app
Copy link

bedevere-app bot commented Dec 2, 2024

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

@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label Dec 2, 2024
@Eclips4
Copy link
Member

Eclips4 commented Dec 2, 2024

Thank you, Bénédikt!

I'd prefer to backport this to 3.12 and 3.13, as these entries are incorrect (and this applies to both 3.12 and 3.13).

Eclips4 pushed a commit that referenced this pull request Dec 2, 2024
…7451) (#127496)

gh-127443: Fix some entries in `Doc/data/refcounts.dat` (GH-127451)

Fix incorrect entries in `Doc/data/refcounts.dat`
(cherry picked from commit 1f8267b)

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
Eclips4 pushed a commit that referenced this pull request Dec 2, 2024
…7451) (#127497)

gh-127443: Fix some entries in `Doc/data/refcounts.dat` (GH-127451)

Fix incorrect entries in `Doc/data/refcounts.dat`
(cherry picked from commit 1f8267b)

Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
@picnixz picnixz deleted the doc/refcounts/fix-syntax-127443 branch December 2, 2024 09:23
picnixz added a commit to picnixz/cpython that referenced this pull request Dec 2, 2024
…#127451)

Fix incorrect entries in `Doc/data/refcounts.dat`
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.

3 participants