-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
gh-118055: Updated the finder glossary entry #118278
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@hugovk , I think this one is good to merge.
The workflow says "waiting to be approved" but I don't see anything wrong, do you agree?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ericsnowcurrently How's this look?
Co-authored-by: Petr Viktorin <[email protected]>
Thanks @Lincoln-developer for the PR, and @encukou for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13. |
(cherry picked from commit db00934) Co-authored-by: Lincoln <[email protected]>
(cherry picked from commit db00934) Co-authored-by: Lincoln <[email protected]>
GH-119773 is a backport of this pull request to the 3.13 branch. |
GH-119774 is a backport of this pull request to the 3.12 branch. |
Thanks @encukou for the merge, let me pick another issue to work on |
The glossary entry predated that language reference page (2012) and importlib (2009), and that no one thought to update the glossary entry at the time. Therefore, this pull request addresses this issue by changing the Python version from 3.3 to 2.3 which PEP 302 targeted. Further by referring to the language reference and the importlib docs as better resources than the PEPs for explaining the various facets of the import system.
📚 Documentation preview 📚: https://cpython-previews--118278.org.readthedocs.build/en/118278/glossary.html#term-finder