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

gh-92417: subprocess docs: remove note on compatibility with Python <3.5 #92538

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AlexWaygood
Copy link
Member

@AlexWaygood AlexWaygood commented May 9, 2022

hugovk
hugovk approved these changes May 9, 2022
Copy link
Member

@hugovk hugovk left a comment

🧹

@@ -33,9 +33,6 @@ The recommended approach to invoking subprocesses is to use the :func:`run`
function for all use cases it can handle. For more advanced use cases, the
underlying :class:`Popen` interface can be used directly.

The :func:`run` function was added in Python 3.5; if you need to retain
Copy link
Contributor

@rhettinger rhettinger May 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We normally keep the notes. on when a function was added.

There are many packages that need to maintain cross-version support even if we don't support the old versions. This information is critical for them.

Copy link
Member Author

@AlexWaygood AlexWaygood May 9, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The .. versionadded:: 3.5 note for subprocess.run(), on line 103 of this file, is not touched by this PR (and I would not want to remove it). This PR proposes to remove an additional note, which seems to me to be overly prominent now that 3.4 has been end-of-life for a long time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants