Skip to content

PEP 798: Fixes and Clarifications from Discourse Thread #4518

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

Merged
merged 7 commits into from
Aug 2, 2025

Conversation

adqm
Copy link
Contributor

@adqm adqm commented Aug 1, 2025

This PR represents a few small wording changes for PEP 798 (primarily in the "How to Teach This" section) based on discussion from the Discourse thread, attempting to fix some correctness issues and to clarify some points. It also fixes syntax highlighting for the examples from other languages.

  • 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--4518.org.readthedocs.build/

@adqm adqm requested a review from JelleZijlstra as a code owner August 1, 2025 16:30
@adqm
Copy link
Contributor Author

adqm commented Aug 1, 2025

@JelleZijlstra, just a heads-up there are likely more changes to be made here (I haven't yet made any changes to address the concerns about regular versus async generators from the Discourse thread, for example, other than adding a link to PEP 525).

@JelleZijlstra JelleZijlstra merged commit b1facf0 into python:main Aug 2, 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.

2 participants