GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — gh-156233: Fix grammar in MRO documentation by aryansk · Pull Request #156320 · python/cpython

Conversation

@aryansk

@aryansk aryansk commented Aug 24, 2026 •
edited by bedevere-app Bot
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

Fixes #156233

Problem

Doc/howto/mro.rst contains a grammar typo:

Then repeat the operation until all the class are removed or it is impossible to find good heads.

Singular "class" disagrees with plural verb "are". The sentence describes removing multiple classes during MRO merge, so plural "classes" is correct.

Issue: #156233
Source: Doc/howto/mro.rst:192 (Doc/howto/mro.html#python-2-3-mro)

Change

  • Fix Doc/howto/mro.rst:192 from all the class are removed → all the classes are removed
  • Keep unrelated cleanup out of this PR

Why this approach

Minimal one-line docs fix matching the issue's suggested correction. No code or behavior change.

Testing

command: git diff --check
result: clean

command: grep -n "all the class" Doc/howto/mro.rst
result: no matches (fixed)

command: grep -n "all the classes are removed" Doc/howto/mro.rst
result: 192:  the operation until all the classes are removed or it is impossible to

Doc build: Doc/howto/mro.rst renders as before with corrected grammar; no functional tests required for docs-only change.

Documentation and release impact

  • User-facing documentation updated
  • Changelog/release note needed: docs fix, no NEWS blurb required per CPython docs-only convention (trivial typo)
  • Migration or compatibility note needed
  • No documentation impact

Review notes

  • Known limitations: none
  • Follow-up issue, if any: none
  • Security/licensing considerations: none
  • Issue: A few typos, grammar mistakes, and code snippet errors in the docs #156233

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
All reactions
@aryansk
pythongh-156233: Fix grammar in MRO documentation
6b7f5b3 
Fixes python#156233

The sentence "Then repeat the operation until all the class are
removed..." incorrectly uses singular "class".
Change to "all the classes are removed" to match plural subject.

Validation: git diff --check clean; verified rendering of
Doc/howto/mro.rst via local inspection.
@bedevere-app bedevere-app Bot added the docs Documentation in the Doc dir label Aug 24, 2026
@python-cla-bot

python-cla-bot Bot commented Aug 24, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@bedevere-app bedevere-app Bot added the skip news label Aug 24, 2026
@github-project-automation github-project-automation Bot added this to Docs PRs Aug 24, 2026
@github-project-automation github-project-automation Bot moved this to Todo in Docs PRs Aug 24, 2026
@bedevere-app bedevere-app Bot mentioned this pull request Aug 24, 2026
A few typos, grammar mistakes, and code snippet errors in the docs #156233
Open
@read-the-docs-community

read-the-docs-community Bot commented Aug 24, 2026

Copy link
Copy Markdown

Documentation build overview

📚 cpython-previews | 🛠️ Build #34207577 | 📁 Comparing 6b7f5b3 against main (ee1da7e)

  🔍 Preview build  

1 file changed
± howto/mro.html
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@willy-b

willy-b commented Aug 27, 2026 •
edited
Loading

Uh oh!

There was an error while loading. Please reload this page.

Copy link
Copy Markdown
Contributor

Thanks very much for sharing this.

However, I think this is now redundant with #156475 (I had already agreed to fix this issue I discovered/reported before you opened this PR) which also fixes another typo and a couple of code snippet errors, so you can probably go ahead and close this PR.

If you would like me, going forward, to provide you suspected typos and other documentation errors to send PRs for and follow to resolution, let me know and I can divvy up what I find in the future and offer to assign some to you.
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@StanFromIreland

StanFromIreland commented Aug 28, 2026

Copy link
Copy Markdown
Member

Closing in favour of #156475
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
@StanFromIreland StanFromIreland closed this Aug 28, 2026
@willy-b

willy-b commented Sep 17, 2026

Copy link
Copy Markdown
Contributor

Btw @aryansk I messaged you on LinkedIn on 2026-09-13 with another typo that you can report and fix if you want (not sure if you check your LinkedIn). I also have another open source issue that I won't be able to get to for several days at least, if you reply to me on LinkedIn so I am allowed to send you another PM, I can send that to you as well. For that one you would be making real code changes in a Python org associated project (no guarantees on whether they would accept them or not).
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

Docs PRs
Status: Todo

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

A few typos, grammar mistakes, and code snippet errors in the docs

Uh oh!

There was an error while loading. Please reload this page.

3 participants

@aryansk @willy-b @StanFromIreland
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.