Skip to content
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

subprocess docs: Fix semantically important typo #119752

Merged
merged 1 commit into from
May 29, 2024

Conversation

zware
Copy link
Member

@zware zware commented May 29, 2024

GH-25416 accidentally replaced a reference to the stderr argument of
subprocess.run with a reference to the stdin argument. stdin is
not affected by the check_output option.


📚 Documentation preview 📚: https://cpython-previews--119752.org.readthedocs.build/

pythonGH-25416 accidentally replaced a reference to the *stderr* argument of
`subprocess.run` with a reference to the *stdin* argument.  *stdin* is
not affected by the `check_output` option.
@zware zware added docs Documentation in the Doc dir skip issue skip news needs backport to 3.11 only security fixes needs backport to 3.12 bug and security fixes needs backport to 3.13 bugs and security fixes labels May 29, 2024
@zware zware self-assigned this May 29, 2024
@zware zware requested a review from erlend-aasland May 29, 2024 21:08
@erlend-aasland
Copy link
Contributor

erlend-aasland commented May 29, 2024

Ouch, thanks a lot for fixing that!

@zware zware enabled auto-merge (squash) May 29, 2024 22:00
@zware zware merged commit 2cc3502 into python:main May 29, 2024
35 checks passed
@zware zware deleted the fix_subprocess_run_doc branch May 29, 2024 22:19
@miss-islington-app
Copy link

Thanks @zware for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11, 3.12, 3.13.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 29, 2024
pythonGH-25416 accidentally replaced a reference to the *stderr* argument of
`subprocess.run` with a reference to the *stdin* argument.  *stdin* is
not affected by the `check_output` option.
(cherry picked from commit 2cc3502)

Co-authored-by: Zachary Ware <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented May 29, 2024

GH-119757 is a backport of this pull request to the 3.13 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 29, 2024
pythonGH-25416 accidentally replaced a reference to the *stderr* argument of
`subprocess.run` with a reference to the *stdin* argument.  *stdin* is
not affected by the `check_output` option.
(cherry picked from commit 2cc3502)

Co-authored-by: Zachary Ware <[email protected]>
@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label May 29, 2024
miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 29, 2024
pythonGH-25416 accidentally replaced a reference to the *stderr* argument of
`subprocess.run` with a reference to the *stdin* argument.  *stdin* is
not affected by the `check_output` option.
(cherry picked from commit 2cc3502)

Co-authored-by: Zachary Ware <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented May 29, 2024

GH-119758 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 bug and security fixes label May 29, 2024
@bedevere-app
Copy link

bedevere-app bot commented May 29, 2024

GH-119759 is a backport of this pull request to the 3.11 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.11 only security fixes label May 29, 2024
AlexWaygood pushed a commit that referenced this pull request May 30, 2024
…119757)

subprocess docs: Fix semantically important typo (GH-119752)

GH-25416 accidentally replaced a reference to the *stderr* argument of
`subprocess.run` with a reference to the *stdin* argument.  *stdin* is
not affected by the `check_output` option.
(cherry picked from commit 2cc3502)

Co-authored-by: Zachary Ware <[email protected]>
AlexWaygood pushed a commit that referenced this pull request May 30, 2024
…119758)

subprocess docs: Fix semantically important typo (GH-119752)

GH-25416 accidentally replaced a reference to the *stderr* argument of
`subprocess.run` with a reference to the *stdin* argument.  *stdin* is
not affected by the `check_output` option.
(cherry picked from commit 2cc3502)

Co-authored-by: Zachary Ware <[email protected]>
noahbkim pushed a commit to hudson-trading/cpython that referenced this pull request Jul 11, 2024
pythonGH-25416 accidentally replaced a reference to the *stderr* argument of
`subprocess.run` with a reference to the *stdin* argument.  *stdin* is
not affected by the `check_output` option.
estyxx pushed a commit to estyxx/cpython that referenced this pull request Jul 17, 2024
pythonGH-25416 accidentally replaced a reference to the *stderr* argument of
`subprocess.run` with a reference to the *stdin* argument.  *stdin* is
not affected by the `check_output` option.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants