Skip to content

Conversation

@davepeck
Copy link
Contributor

@davepeck davepeck commented Apr 7, 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)

As part of pre-acceptance of PEP 750, the Steering Council requested four small updates to the PEP. I've taken a first cut at all four:

  • Specify final module location as string.templatelib
  • Spell out conv as conversion and expr as expression
  • Provide several examples of Template.__iter__()
  • Be more explicit in the section on the inability to round-trip source text
  • Possibly spell out format_spec as format_specification (TBD)
  • Edit and correct these changes

With this PR, hopefully we can all put eyes on PEP 750 and get it to a state we're excited about! 🎉


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

davepeck and others added 2 commits April 7, 2025 11:21
Apply several of @AA-Turner's suggestions. Will address the remaining separately.

Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
@Wingysam
Copy link

Wingysam commented Apr 7, 2025

Looks like you beat me to this! I have my own branch that does some of this locally and just saw that you've finished doing it haha.

The interpreter implementation still needs the update. I'll continue working on my fork.

@davepeck
Copy link
Contributor Author

davepeck commented Apr 7, 2025

@AA-Turner Thanks for the suggestions -- took a swing at all of them! :-)

@AA-Turner
Copy link
Member

Thank you!

A

Copy link
Member

@lysnikolaou lysnikolaou left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks @davepeck!

@hugovk hugovk merged commit 9b68bf5 into python:main Apr 9, 2025
5 checks passed
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.

5 participants