Skip to content

PEP 823: Expand 'How to Teach This' to a mini-tutorial - #5177

Merged
gvanrossum merged 8 commits into
python:mainfrom
gvanrossum:teach823
Oct 20, 2026
Merged

gvanrossum merged 8 commits into
python:mainfrom
gvanrossum:teach823

Conversation

@gvanrossum

@gvanrossum gvanrossum commented Oct 19, 2026 •

Copy link
Copy Markdown
Member

Don't merge until @cdce8p (Marc Müller, the PEP author) has approved this.

Copilot AI left a comment

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.

🟢 Approval recommended

The documentation-only changes explain the proposed behavior consistently, with no blocking issues identified.

No open issues 🎉

What changed in this PR

Expands PEP 823’s teaching guidance into a mini-tutorial for the proposed None-aware operators.

Changes:

  • Adds prerequisites, examples, chaining rules, and caveats.
  • Explains when to use—and avoid—the operators.
  • Removes an outdated cross-reference to the teaching section.
File Description
peps/​pep-0823.rst Expands teaching guidance and adjusts related explanatory text.

Review effort: Balanced

@read-the-docs-community

read-the-docs-community Bot commented Oct 19, 2026 •

Copy link
Copy Markdown

@cdce8p cdce8p left a comment

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.

Thanks for putting this together! I agree let's include it, it's a much better use for that section. Left some minor comments.

Comment thread peps/pep-0823.rst Outdated
Comment thread peps/pep-0823.rst
Comment thread peps/pep-0823.rst Outdated
Comment thread peps/pep-0823.rst
Comment thread peps/pep-0823.rst Outdated
@cdce8p

cdce8p commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

It might also be good to add a short note to the (to be created) Change History section (after the footnotes).

@gvanrossum
gvanrossum merged commit b5125a6 into python:main Oct 20, 2026
5 checks passed
@gvanrossum
gvanrossum deleted the teach823 branch October 20, 2026 17:20
@gvanrossum

Copy link
Copy Markdown
Member Author

@cdce8p If you have any concerns with my changes in response to your feedback, let's open a new PR to address those concerns.

@cdce8p

cdce8p commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Nothing more to add here. Thanks again @gvanrossum!

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