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

bpo-40707: Document that Popen.communicate sets the returncode attribute #20283

Merged
merged 3 commits into from Jun 24, 2020

Conversation

gareth-rees
Copy link
Contributor

@gareth-rees gareth-rees commented May 21, 2020

This pull request fixes issue 40707 by documenting that the subprocess.Popen.communicate() method sets the returncode attribute.

https://bugs.python.org/issue40707

Copy link
Contributor

@remilapeyre remilapeyre left a comment

Thanks for improving the docs @gareth-rees!

Doc/library/subprocess.rst Outdated Show resolved Hide resolved
Co-authored-by: Rémi Lapeyre <remi.lapeyre@henki.fr>
@csabella csabella requested a review from giampaolo May 23, 2020
Doc/library/subprocess.rst Outdated Show resolved Hide resolved
@terryjreedy terryjreedy merged commit bf2e515 into python:master Jun 24, 2020
4 checks passed
@miss-islington
Copy link
Contributor

miss-islington commented Jun 24, 2020

Thanks @gareth-rees for the PR, and @terryjreedy for merging it 🌮🎉.. I'm working now to backport this PR to: 3.8, 3.9.
🐍🍒🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 24, 2020
…ute (pythonGH-20283)

(cherry picked from commit bf2e515)

Co-authored-by: Gareth Rees <gdr@garethrees.org>
@bedevere-bot
Copy link

bedevere-bot commented Jun 24, 2020

GH-21105 is a backport of this pull request to the 3.9 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 24, 2020
…ute (pythonGH-20283)

(cherry picked from commit bf2e515)

Co-authored-by: Gareth Rees <gdr@garethrees.org>
@bedevere-bot
Copy link

bedevere-bot commented Jun 24, 2020

GH-21106 is a backport of this pull request to the 3.8 branch.

miss-islington added a commit that referenced this pull request Jun 24, 2020
…ute (GH-20283)

(cherry picked from commit bf2e515)

Co-authored-by: Gareth Rees <gdr@garethrees.org>
miss-islington added a commit that referenced this pull request Jun 24, 2020
…ute (GH-20283)

(cherry picked from commit bf2e515)

Co-authored-by: Gareth Rees <gdr@garethrees.org>
fasih pushed a commit to fasih/cpython that referenced this pull request Jun 29, 2020
arun-mani-j pushed a commit to arun-mani-j/cpython that referenced this pull request Jul 21, 2020
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 news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants