Skip to content

DRAFT: Implement PEP 802 and the {/} notation #137565

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.

Sign up for GitHub

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

Jump to bottom
Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

DRAFT: Implement PEP 802 and the {/} notation #137565

wants to merge 2 commits into from

Conversation

AA-Turner
Copy link
Member

@AA-Turner AA-Turner commented Aug 8, 2025 •
edited by github-actions bot
Loading

This is a draft proof-of-concept implementation of PEP 802.

A

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

nineteendo
nineteendo reviewed Aug 8, 2025
View reviewed changes
Copy link
Contributor

@nineteendo nineteendo left a comment

Choose a reason for hiding this comment

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

I think you forgot Lib/pprint.py.

@AA-Turner
Copy link
Member Author

@nineteendo can you elaborate?

Running Release|x64 interpreter...
Python 3.15.0a0 (remotes/upstream/HEAD-1-g71075ef1c9e-dirty:71075ef1c9e, Aug  8 2025, 19:28:32) [MSC v.1942 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from pprint import pp
>>> pp({/})
{/}
>>> pp(set())
{/}
>>> pp(frozenset())
frozenset()

@nineteendo
Copy link
Contributor

nineteendo commented Aug 9, 2025 •
edited
Loading

Ah, that's handled by repr(), I didn't need to change it:

cpython/Lib/pprint.py

Lines 312 to 313 in d7dbde8

if not len(object):
stream.write(repr(object))

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nineteendo nineteendo nineteendo left review comments

@willingc willingc Awaiting requested review from willingc willingc will be requested when the pull request is marked ready for review willingc is a code owner

@Eclips4 Eclips4 Awaiting requested review from Eclips4 Eclips4 will be requested when the pull request is marked ready for review Eclips4 is a code owner

@tomasr8 tomasr8 Awaiting requested review from tomasr8 tomasr8 will be requested when the pull request is marked ready for review tomasr8 is a code owner

@isidentical isidentical Awaiting requested review from isidentical isidentical will be requested when the pull request is marked ready for review isidentical is a code owner

@JelleZijlstra JelleZijlstra Awaiting requested review from JelleZijlstra JelleZijlstra will be requested when the pull request is marked ready for review JelleZijlstra is a code owner

@pablogsal pablogsal Awaiting requested review from pablogsal pablogsal will be requested when the pull request is marked ready for review pablogsal is a code owner

@lysnikolaou lysnikolaou Awaiting requested review from lysnikolaou lysnikolaou will be requested when the pull request is marked ready for review lysnikolaou is a code owner

@rhettinger rhettinger Awaiting requested review from rhettinger rhettinger will be requested when the pull request is marked ready for review rhettinger is a code owner

Assignees
No one assigned
Labels
DO-NOT-MERGE
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@AA-Turner @nineteendo