-
-
Notifications
You must be signed in to change notification settings - Fork 30.1k
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
shutil.which()
docs reference os.environ as :data:
#124494
Conversation
And make use of :envvar: when talking about PATH and PATHEXT.
Thanks @twm for the PR, and @AA-Turner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12, 3.13. |
…h()`` (pythonGH-124494) (cherry picked from commit 0d38409) Co-authored-by: Tom Most <[email protected]>
GH-124497 is a backport of this pull request to the 3.13 branch. |
Sorry, @twm and @AA-Turner, I could not cleanly backport this to
|
Thanks Tom, are you able to handle backports to 3.12? Let me know if you have any problems with A |
@AA-Turner Sure, I will give it a try this evening! |
…il.which()`` (GH-124494) (#124497) Doc: Improve documentation for the ``path`` argument in ``shutil.which()`` (GH-124494) (cherry picked from commit 0d38409) Co-authored-by: Tom Most <[email protected]>
GH-124576 is a backport of this pull request to the 3.12 branch. |
GH-124576 is a backport of this pull request to the 3.12 branch. |
The
shutil.which()
docs referencedos.environ
using:func:`os.environ`
, which renders as "os.environ()" — nonsensical, as the mapping is not callable. The correct notation is:data:`os.environ`
.I also made use of
:envvar:
when talking about PATH and PATHEXT, following the pattern in the venv docs.I did not file an issue, nor create a blurb (the previous PR here didn't), as this change seems trivial. Happy to do so, though! I did reword the paragraph a bit, though most of the diff is caused by re-wrapping lines.
📚 Documentation preview 📚: https://cpython-previews--124494.org.readthedocs.build/en/124494/library/shutil.html