Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

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

Provide feedback

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

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

gh-54732: Make argparse error caused by empty rows in option files explicit#136795

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
AA-Turner merged 6 commits intopython:mainfromjdunter:doc_argparse_empty_rows
Jul 19, 2025

Conversation

jdunter
Copy link
Contributor

@jdunterjdunter commentedJul 19, 2025•
edited by bedevere-appbot
Loading

Adds a Note to the documentation, that addresses the error caused by an empty row in an argparse option file.

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

@python-cla-bot
Copy link

python-cla-botbot commentedJul 19, 2025•
edited
Loading

All commit authors signed the Contributor License Agreement.

CLA signed

@bedevere-appbedevere-appbot added awaiting review docsDocumentation in the Doc dir skip news labelsJul 19, 2025
@StanFromIrelandStanFromIreland changed the titleAddresses #54732: Make argparse error caused by empty rows in option files explicit.gh-54732: Make argparse error caused by empty rows in option files explicit.Jul 19, 2025
@StanFromIreland
Copy link
Member

Please use the format you have in the issue description for the title, it allows our automation to link it. You also need to sign the CLA.

jdunter reacted with thumbs up emoji

@jdunterjdunterforce-pushed thedoc_argparse_empty_rows branch fromd2dd1e9 to1cfa51bCompareJuly 19, 2025 11:53
@StanFromIreland
Copy link
Member

StanFromIreland commentedJul 19, 2025•
edited
Loading

Please do not use theUpdate Branch button unless necessary (e.g. fixing conflicts, jogging the CI, or very old PRs) as it uses valuable resources. For more information see thedevguide.

jdunter reacted with thumbs up emoji

Namely the use of code markup.Also I fixed a trailing whitespace.
@AA-TurnerAA-Turner added needs backport to 3.13bugs and security fixes needs backport to 3.14bugs and security fixes labelsJul 19, 2025
@AA-TurnerAA-Turner changed the titlegh-54732: Make argparse error caused by empty rows in option files explicit.gh-54732: Make argparse error caused by empty rows in option files explicitJul 19, 2025
Change two of the phrasings to make it more clear.Use straight quotes.
@AA-TurnerAA-Turnerenabled auto-merge (squash)July 19, 2025 14:03
@AA-TurnerAA-Turner merged commit8ffc3ef intopython:mainJul 19, 2025
31 of 32 checks passed
@miss-islington-app
Copy link

Thanks@jdunter for the PR, and@AA-Turner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestJul 19, 2025
…les explicit (pythonGH-136795)(cherry picked from commit 8ffc3ef01e83ffe629c6107082677de4d23974d5)Co-authored-by: jdunter <2ve@mailbox.org>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
miss-islington pushed a commit to miss-islington/cpython that referenced this pull requestJul 19, 2025
…les explicit (pythonGH-136795)(cherry picked from commit 8ffc3ef01e83ffe629c6107082677de4d23974d5)Co-authored-by: jdunter <2ve@mailbox.org>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
@bedevere-app
Copy link

GH-136818 is a backport of this pull request to the3.14 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.14bugs and security fixes labelJul 19, 2025
@bedevere-app
Copy link

GH-136819 is a backport of this pull request to the3.13 branch.

@bedevere-appbedevere-appbot removed the needs backport to 3.13bugs and security fixes labelJul 19, 2025
AA-Turner added a commit that referenced this pull requestJul 19, 2025
…iles explicit (GH-136795) (#136818)gh-54732: Make argparse error caused by empty rows in option files explicit (GH-136795)(cherry picked from commit8ffc3ef)Co-authored-by: jdunter <2ve@mailbox.org>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
AA-Turner added a commit that referenced this pull requestJul 19, 2025
…iles explicit (GH-136795) (#136819)gh-54732: Make argparse error caused by empty rows in option files explicit (GH-136795)(cherry picked from commit8ffc3ef)Co-authored-by: jdunter <2ve@mailbox.org>Co-authored-by: Adam Turner <9087854+AA-Turner@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@StanFromIrelandStanFromIrelandStanFromIreland left review comments

@AA-TurnerAA-TurnerAA-Turner approved these changes

@savannahostrowskisavannahostrowskiAwaiting requested review from savannahostrowskisavannahostrowski is a code owner

Assignees
No one assigned
Labels
docsDocumentation in the Doc dirskip newssprint
Projects
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@jdunter@StanFromIreland@AA-Turner
[8]ページ先頭
©2009-2025 Movatter.jp