Skip to content

PEP 824: Expand 'How to Teach This' to a mini-tutorial - #5179

Merged
gvanrossum merged 6 commits into
python:mainfrom
gvanrossum:teach824
Oct 20, 2026
Merged

gvanrossum merged 6 commits into
python:mainfrom
gvanrossum:teach824

Conversation

@gvanrossum

Copy link
Copy Markdown
Member

Don't merge until @cdce8p (Marc Mueller, the PEP author) approves!

@read-the-docs-community

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

Copy link
Copy Markdown

Documentation build overview

📚 pep-previews | 🛠️ Build #35277358 | 📁 Comparing 9926a72 against latest (d5d2571)

  🔍 Preview build  

1 file changed
± pep-0824/index.html

@willingc willingc 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 @gvanrossum. Chaining and operator precedence sections are very helpful and descriptive. Leaving for the PEP author to weigh in prior to merging.

@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.

Similar to the other PR, it could be nice to add a short note to the (to be created) Change History section.

Comment thread peps/pep-0824.rst Outdated
Comment thread peps/pep-0824.rst Outdated
Comment thread peps/pep-0824.rst Outdated

@gvanrossum gvanrossum left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

LGTM

@gvanrossum
gvanrossum merged commit 1317769 into python:main Oct 20, 2026
5 checks passed
@gvanrossum
gvanrossum deleted the teach824 branch October 20, 2026 18:03
@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

Same here, nothing more to add. The rewritten d.get(key) paragraph is clearer now.

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