Skip to content

Missing subcommand no longer prints all help text#776

Merged
tleonhardt merged 1 commit intomasterfrom
required_subcmd
Sep 18, 2019
Merged

Missing subcommand no longer prints all help text#776
tleonhardt merged 1 commit intomasterfrom
required_subcmd

Conversation

@kmvanbrunt
Copy link
Copy Markdown
Member

No longer printing all help text for alias and macro when the subcommand is omitted. Instead allow argparse to handle the error.

This was done to simplify our code and make error handling consistent across commands.

…and is omitted. Instead allow argparse to handle the error.
@kmvanbrunt kmvanbrunt added this to the 0.9.17 milestone Sep 17, 2019
@kmvanbrunt kmvanbrunt self-assigned this Sep 17, 2019
Copy link
Copy Markdown
Member

@tleonhardt tleonhardt left a comment

Choose a reason for hiding this comment

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

Looks good

@tleonhardt tleonhardt merged commit 1176c0c into master Sep 18, 2019
@tleonhardt tleonhardt deleted the required_subcmd branch September 18, 2019 01:26
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.

2 participants