◐ Shell
reader mode source ↗
Skip to content

bpo-17050: Remove documentation on argparse.REMAINDER#18661

Merged
miss-islington merged 1 commit into
python:masterfrom
corollari:master
May 24, 2020
Merged

bpo-17050: Remove documentation on argparse.REMAINDER#18661
miss-islington merged 1 commit into
python:masterfrom
corollari:master

Conversation

@corollari

@corollari corollari commented Feb 25, 2020 •
edited by miss-islington
Loading

Copy link
Copy Markdown
Contributor

Closes Issue 17050 by removing argparse.REMAINDER from the documentation, as discussed on the issue.

https://bugs.python.org/issue17050

Automerge-Triggered-By: @rhettinger

@the-knights-who-say-ni

the-knights-who-say-ni commented Feb 25, 2020

Copy link
Copy Markdown

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept this contribution by verifying everyone involved has signed the PSF contributor agreement (CLA).

Recognized GitHub username

We couldn't find a bugs.python.org (b.p.o) account corresponding to the following GitHub usernames:

@corollari

This might be simply due to a missing "GitHub Name" entry in one's b.p.o account settings. This is necessary for legal reasons before we can look at this contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for the contribution, we look forward to reviewing it!

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting review labels Feb 25, 2020
@corollari

corollari commented Feb 25, 2020

Copy link
Copy Markdown
Contributor Author

I've already signed the CLA and I'm currently just waiting for it to get accepted.

Regarding the bedevere/news CI failure, I'm a bit confused by it because the docs on news blurbs states that "Changes that affect documentation only generally do not require a news entry.". Is there something I'm missing?

eamanu
eamanu approved these changes Feb 26, 2020
View reviewed changes

@eamanu eamanu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hide comment

LGTM.

@eamanu

eamanu commented Feb 26, 2020

Copy link
Copy Markdown
Contributor

@corollari yes you are right, this PR does not need a NEWs.

@csabella csabella requested a review from rhettinger May 22, 2020 20:05
@miss-islington

miss-islington commented May 24, 2020

Copy link
Copy Markdown
Contributor

@corollari: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 59f5022 into python:master May 24, 2020
@miss-islington

miss-islington commented May 24, 2020

Copy link
Copy Markdown
Contributor

Thanks @corollari for the PR 🌮🎉.. I'm working now to backport this PR to: 3.9.
🐍🍒⛏🤖

@bedevere-bot

bedevere-bot commented May 24, 2020

Copy link
Copy Markdown

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

rhettinger pushed a commit that referenced this pull request May 24, 2020
@miss-islington
bpo-17050: Remove documentation on argparse.REMAINDER (GH-18661) (GH-… …
46c1b9c 
…20363)
@sobolevn sobolevn mentioned this pull request Apr 17, 2023
Merged
@hauntsaninja hauntsaninja mentioned this pull request Apr 19, 2023
Open
@mihaic mihaic mentioned this pull request Oct 20, 2023
Closed
@clytaemnestra clytaemnestra mentioned this pull request Jun 1, 2026
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rhettinger rhettinger rhettinger approved these changes

+1 more reviewer

@eamanu eamanu eamanu approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@corollari @the-knights-who-say-ni @eamanu @miss-islington @bedevere-bot @rhettinger