Skip to content

gh-142461: Move misplaced NEWS entries to an appropriate section#143411

Merged
serhiy-storchaka merged 7 commits into
python:mainfrom
Aniketsy:fix-142461-clean
Jan 12, 2026
Merged

gh-142461: Move misplaced NEWS entries to an appropriate section#143411
serhiy-storchaka merged 7 commits into
python:mainfrom
Aniketsy:fix-142461-clean

Conversation

@Aniketsy

@Aniketsy Aniketsy commented Jan 4, 2026

Copy link
Copy Markdown
Contributor

This includes changes in 3.10 and 3.11.

@StanFromIreland

Copy link
Copy Markdown
Member

Update your branch to fix the CI.

@picnixz

picnixz commented Jan 4, 2026

Copy link
Copy Markdown
Member

There are wrong changes. Zip is a built-in for instance. The ast module is a core module IIRC.

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

Core and Builtins also include built-in types (e.g., zip, int, map, etc), and their methods. I won't go through all occurrences, but please correct them.

Comment thread Misc/NEWS.d/3.11.0a5.rst Outdated
.. date: 2021-05-04-21-55-49
.. nonce: M9m8Qd
.. section: Core and Builtins
.. section: Library

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.

This must stay as a core and builtins. It's about getattr/hasattr which are builtins.

This comment was marked as spam.

@bedevere-app

bedevere-app Bot commented Jan 4, 2026

Copy link
Copy Markdown

A Python core developer has requested some changes be made to your pull request before we can consider merging it. If you could please address their requests along with any other requests in other reviews from core developers that would be appreciated.

Once you have made the requested changes, please leave a comment on this pull request containing the phrase I have made the requested changes; please review again. I will then notify any core developers who have left a review that you're ready for them to take another look at this pull request.

@Aniketsy

Aniketsy commented Jan 4, 2026

Copy link
Copy Markdown
Contributor Author

Thanks, I’ll revisit these and also correct them in the previous PR

Comment thread Misc/NEWS.d/3.10.0a2.rst
.. section: Core and Builtins
.. section: Library

Add ``sys._current_exceptions()`` function to retrieve a dictionary mapping

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.

sysmodule is a core & built-in strictly speaking, but I think it's fine to keep it under Library as it's the user-facing stuff.

This comment was marked as spam.

Comment thread Misc/NEWS.d/3.10.0a5.rst Outdated
.. date: 2021-01-13-14-06-01
.. nonce: _WS1Ok
.. section: Core and Builtins
.. section: Library

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.

bytearray is a built-in...

Comment thread Misc/NEWS.d/3.10.0a5.rst Outdated
.. date: 2018-12-20-23-59-23
.. nonce: idHEcj
.. section: Core and Builtins
.. section: Library

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.

This is affecting string formattin... so built-in.

Comment thread Misc/NEWS.d/3.10.0a7.rst
Comment thread Misc/NEWS.d/3.10.0a7.rst Outdated
.. date: 2021-03-16-17-12-54
.. nonce: zAo6Ws
.. section: Core and Builtins
.. section: Library

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.

Again.

Comment thread Misc/NEWS.d/3.11.0a1.rst Outdated
.. date: 2021-09-02-01-28-01
.. nonce: QDjM_l
.. section: Core and Builtins
.. section: Library

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.

I'd say both Library and Core and Builtins. I'd prefer Core and Builtins because of open.

Comment thread Misc/NEWS.d/3.11.0a5.rst Outdated
.. date: 2022-01-25-19-34-55
.. nonce: mQLNPk
.. section: Core and Builtins
.. section: Library

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.

Builtin.

Comment thread Misc/NEWS.d/3.11.0b1.rst
@Aniketsy

Aniketsy commented Jan 4, 2026

Copy link
Copy Markdown
Contributor Author

I’d like to apologize for increasing the review workload.

Comment thread Misc/NEWS.d/3.10.0a5.rst Outdated
.. date: 2020-10-10-14-16-03
.. nonce: Xop8sV
.. section: Core and Builtins
.. section: Library

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.

This is still a core & builtins.

Comment thread Misc/NEWS.d/3.10.0a7.rst
.. section: Core and Builtins
.. section: Library

Raise ImportWarning when calling find_loader().

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.

Is this change affecting more only importlib or is it affecting the entire interpreter?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This change only affects importlib

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm thinking this one is Core and Builtins. It's true that the changes are in importlib, but the PR modifies _bootstrap_external.py and thus affects how cpython finds modules globally. Specifically, I expect anything that uses find_loader() would be implicated by this change.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Thanks for pointing this out.

Comment thread Misc/NEWS.d/3.11.0a5.rst Outdated
.. date: 2021-12-11-11-36-48
.. nonce: sfThay
.. section: Core and Builtins
.. section: Library

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.

This one is also a build. I said on the issue that one should be careful about possible duplicated NEWS entries because of backports.

@serhiy-storchaka serhiy-storchaka self-requested a review January 9, 2026 09:33

@serhiy-storchaka serhiy-storchaka 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.

Thank you for your PR @Aniketsy, great work. Mostly LGTM, I only left few minor comments.

Comment thread Misc/NEWS.d/3.10.0a1.rst Outdated
@@ -643,7 +643,7 @@ Convert :mod:`!_bz2` to use :c:func:`PyType_FromSpec`.
.. bpo: 41006
.. date: 2020-06-18-00-07-09
.. nonce: H-wN-d
.. section: Core and Builtins
.. section: Library

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.

I would leave this as Core and Builtins. It is about the behavior of the core interpreter.

Comment thread Misc/NEWS.d/3.10.0a3.rst
Comment on lines +92 to 94
.. section: Library

Port the ``_warnings`` extension module to the multi-phase initialization

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.

This is on border. I have no strong opinion.

There is a stable C API for warnings, but it can be used without importing the module, and I think this change did not affect the users of this C API. So, I think it is fine to keep Library.

Comment thread Misc/NEWS.d/3.11.0a1.rst
@@ -917,7 +917,7 @@ Add ``__parameters__`` attribute and ``__getitem__`` operator to
.. bpo: 44523
.. date: 2021-06-29-11-49-29
.. nonce: 67-ZIP
.. section: Core and Builtins
.. section: Library

Remove the pass-through for :func:`hash` of :class:`weakref.proxy` objects
to prevent unintended consequences when the original referred object dies

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.

There is a C API for weakref.proxy, it is always present. I think it is closer to core.

Comment thread Misc/NEWS.d/3.11.0b1.rst Outdated
@@ -434,7 +434,7 @@ cache when applicable.
.. bpo: 46962
.. date: 2022-03-08-21-59-57
.. nonce: UomDfz
.. section: Core and Builtins
.. section: Library

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.

types.GenericAlias and types.UnionType are builtins. It is okay to keep Core and Builtins.

Or you can split the entry on two. There was already other entry for documentation changes. https://github.com/python/cpython/pull/31769/changes

Comment thread Misc/NEWS.d/3.11.0b1.rst Outdated
@@ -519,7 +519,7 @@ less or equal than the starting position of non-encodable characters.
.. bpo: 34093
.. date: 2018-07-14-16-58-00
.. nonce: WaVD-f
.. section: Core and Builtins
.. section: Library

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.

Marshalling is the part of the core.

And the purpose of this change to make .pyc files more stable.

Comment thread Misc/NEWS.d/3.10.0a4.rst
@@ -102,7 +102,7 @@ blocks
.. bpo: 42639
.. date: 2020-12-09-01-55-10
.. nonce: 5pI5HG
.. section: Core and Builtins
.. section: Library

Make the :mod:`atexit` module state per-interpreter. It is now safe have
more than one :mod:`atexit` module instance. Patch by Donghee Na and Victor

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.

There is no stable public C API for atexit. Even if this is tied to interpreter internals, the only way it can be used by importing the atexit module. So, Library.

@Aniketsy

Aniketsy commented Jan 9, 2026

Copy link
Copy Markdown
Contributor Author

@serhiy-storchaka Thanks for the review. I’ve applied the changes based on your suggestions.

@serhiy-storchaka serhiy-storchaka 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.

LGTM. 👍

@StanFromIreland

Copy link
Copy Markdown
Member

You need to update your branch to fix the CI.

@serhiy-storchaka

Copy link
Copy Markdown
Member

@picnixz, have your comments been addressed?

@picnixz

picnixz commented Jan 12, 2026

Copy link
Copy Markdown
Member

Yes they were. Thank you!

@serhiy-storchaka serhiy-storchaka merged commit 3d44f0a into python:main Jan 12, 2026
28 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Jan 12, 2026
@serhiy-storchaka serhiy-storchaka added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes labels Jan 12, 2026
@miss-islington-app

Copy link
Copy Markdown

Thanks @Aniketsy for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

@miss-islington-app

Copy link
Copy Markdown

Thanks @Aniketsy for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 12, 2026
pythonGH-143411)

(cherry picked from commit 3d44f0a)

Co-authored-by: Aniket <148300120+Aniketsy@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jan 12, 2026
pythonGH-143411)

(cherry picked from commit 3d44f0a)

Co-authored-by: Aniket <148300120+Aniketsy@users.noreply.github.com>
@bedevere-app

bedevere-app Bot commented Jan 12, 2026

Copy link
Copy Markdown

GH-143745 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 Jan 12, 2026
@bedevere-app

bedevere-app Bot commented Jan 12, 2026

Copy link
Copy Markdown

GH-143746 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Jan 12, 2026
serhiy-storchaka pushed a commit that referenced this pull request Jan 12, 2026
…on (GH-143411) (GH-143746)

(cherry picked from commit 3d44f0a)

Co-authored-by: Aniket <148300120+Aniketsy@users.noreply.github.com>
serhiy-storchaka pushed a commit that referenced this pull request Jan 12, 2026
…on (GH-143411) (GH-143745)

(cherry picked from commit 3d44f0a)

Co-authored-by: Aniket <148300120+Aniketsy@users.noreply.github.com>
thunder-coding pushed a commit to thunder-coding/cpython that referenced this pull request Feb 15, 2026
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 news

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

6 participants