FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

PEP 823: Expand 'How to Teach This' to a mini-tutorial by gvanrossum · Pull Request #5177 · python/peps · GitHub

Repository navigation

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 commented Oct 19, 2026 •
edited
Loading

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

🟢 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 Bot commented Oct 19, 2026 •
edited
Loading

Copy link
Copy Markdown

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.

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

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 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 merged commit b5125a6 into python:main Oct 20, 2026
5 checks passed
gvanrossum deleted the teach823 branch October 20, 2026 17:20

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 commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

Nothing more to add here. Thanks again @gvanrossum!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL