Skip to content

Conversation

@skirpichev
Copy link
Contributor

@skirpichev skirpichev commented Sep 14, 2025

  • Change is either:
    • To a Draft PEP
    • To an Accepted or Final PEP, with Steering Council approval
    • To fix an editorial issue (markup, typo, link, header, etc)
  • PR title prefixed with PEP number (e.g. PEP 123: Summary of changes)

📚 Documentation preview 📚: https://pep-previews--4586.org.readthedocs.build/

https://pep-previews--4586.org.readthedocs.build/pep-0791/

@skirpichev skirpichev marked this pull request as ready for review September 14, 2025 11:36
@skirpichev
Copy link
Contributor Author

This is a final version, before submitting to SC.

CC @serhiy-storchaka

Copy link
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

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

LGTM

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@vstinner
Copy link
Member

@hugovk: Are you ok with the updated PR?

@skirpichev skirpichev requested a review from hugovk September 17, 2025 12:12
@skirpichev
Copy link
Contributor Author

@vstinner, feel free to apply remaining suggestions.

Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
@vstinner vstinner merged commit 96bb9f2 into python:main Sep 18, 2025
5 checks passed
@vstinner
Copy link
Member

I applied the last sugestion and merged the PR.

@skirpichev skirpichev deleted the pep791-edits branch September 18, 2025 13:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants