gh-125904: Fix MRO summary description in tutorial #125906
Closed
Conversation
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
The tutorial classes chapter multiple inheritance section described MRO as depth first. That was the case for the old classic classes in Python2.
|
All commit authors signed the Contributor License Agreement. |
bedevere-app
bot
added
docs
ZeroIntensity
added
needs backport to 3.12
ZeroIntensity
approved these changes
Oct 24, 2024
erlend-aasland
changed the title
|
|
||
| For most purposes, in the simplest cases, you can think of the search for | ||
| attributes inherited from a parent class as depth-first, left-to-right, not | ||
| attributes inherited from a parent class as breadth-first, left-to-right, not |
There was a problem hiding this comment.
What about the following example in L643-L645?
There was a problem hiding this comment.
What about the following example in L643-L645?
Good point. Have been thinking about this more. Maybe should cancel the PR and discuss this portion of the docs further in the issue ticket #125904?
|
Cancelling this request. Leaving issue #125904 open to consider further how to update this section. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
The tutorial classes chapter multiple inheritance section described MRO as depth first. That was the case for the old classic classes in Python2.
📚 Documentation preview 📚: https://cpython-previews--125906.org.readthedocs.build/