← 返回首页
gh-93240: clarify wording in IO tutorial by paulreece · Pull Request #93276 · python/cpython · GitHub
Skip to content

Navigation Menu

Toggle navigation
Sign in
Appearance settings
Search or jump to...

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Include my email address so I can be contacted

Saved searches

Use saved searches to filter your results more quickly

Appearance settings
Resetting focus

gh-93240: clarify wording in IO tutorial#93276

Merged
ambv merged 2 commits into
python:mainfrom
paulreece:docdicttablefix
Jun 21, 2022
Merged

gh-93240: clarify wording in IO tutorial#93276
ambv merged 2 commits into
python:mainfrom
paulreece:docdicttablefix

Conversation

paulreece commented May 27, 2022
edited
Loading

Copy link
Copy Markdown
Contributor

Changed the wording of the Docs to "This could also be done by passing the table dictionary as keyword arguments with the ** notation. " on line 182 of the inputoutpout.rst file.

ghost commented May 27, 2022
edited by ghost
Loading

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

Copy link
Copy Markdown

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

AA-Turner commented May 27, 2022
edited
Loading

Copy link
Copy Markdown
Member

Please update the PR title to include the issue number (gh-93240: clarify wording in IO tutorial).

You don't need a NEWS entry for this.

A

Comment thread Doc/tutorial/inputoutput.rst Outdated Show resolved Hide resolved
AA-Turner added needs backport to 3.10 only security fixes needs backport to 3.11 only security fixes labels May 27, 2022
paulreece changed the title changed the inputoutput docs to reflect the more specific wording men… gh-93240: clarify wording in IO tutorial May 27, 2022

Copy link
Copy Markdown
Contributor Author

Adam, thanks for your help there. I changed the title and removed the news entry from the comment.

Copy link
Copy Markdown
Contributor Author

@rhettinger Please let me know if there is anything else I need to do for this PR in order to get it approved. I ran the checks again and it looks like it passed this time, though it did skip 7 of them. Thanks for your time!

paulreece and others added 2 commits June 13, 2022 10:36
Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
ambv merged commit 51d6731 into python:main Jun 21, 2022

Copy link
Copy Markdown
Contributor

Thanks @paulreece for the PR, and @ambv for merging it 🌮🎉.. I'm working now to backport this PR to: 3.10, 3.11.
🐍🍒⛏🤖

Copy link
Copy Markdown
Contributor

Sorry @paulreece and @ambv, I had trouble checking out the 3.11 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 51d673176ac90f2144e3e7a492e5b3ea53bdba2e 3.11

Copy link
Copy Markdown

GH-94081 is a backport of this pull request to the 3.10 branch.

bedevere-bot removed the needs backport to 3.10 only security fixes label Jun 21, 2022

Copy link
Copy Markdown

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

bedevere-bot removed the needs backport to 3.11 only security fixes label Jun 21, 2022

Copy link
Copy Markdown
Contributor Author

@miss-islington Thank you so much for the warm message, I am so thrilled to have officially contributed to Python!

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

6 participants

Footer

© 2026 GitHub, Inc.