Skip to content

Conversation

@ppaez
Copy link
Contributor

@ppaezppaez commented Jan 31, 2023

This provides a fix to improve the documentation.

@hpaulj
Copy link

This patch conflicts with a more recent one, #103582

argparse.REMAINDER is ...

We can't both remove all references to REMAINDER and add it's literal value back in.

Subparsers use 'argparse.PARSERwhich is 'A...'. It's a validnargsvalue, but only used by the_SubParsersActionclass. Not all possiblenargs` values have to be documented.

@hauntsaninja
Copy link
Contributor

hauntsaninja commented Apr 19, 2023

Thanks for the PR, and thanks hpaulj for linking to the other PR.

However, I left a comment on #101472 (comment) mentioning what I think the ways forward for this are.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting reviewdocsDocumentation in the Doc dirskip news

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@ppaez@hpaulj@hauntsaninja@bedevere-bot