Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Example for an external subcommand. #1012

Closed
wants to merge 1 commit into from

Conversation

marmistrz
Copy link

@marmistrz marmistrz commented Jul 24, 2017

Add an example for spawning an external subcommand with a multiply existing argument. This was pretty hard for me to find.

Why is the subcommand not displayed in help? I get:

app [OPTIONS]

I'd expect

app [OPTIONS] [COMMAND]

This change is Reviewable

@coveralls
Copy link

coveralls commented Jul 24, 2017

Coverage Status

Coverage remained the same at 91.929% when pulling ee2f5ec on marmistrz:master into 76869a0 on kbknapp:master.

@kbknapp
Copy link
Member

kbknapp commented Jul 29, 2017

I think it'd be good to add another statement down below the println showing how to use a match and pull those "additional subcommand" args out of the grep external subcommand.

@marmistrz
Copy link
Author

marmistrz commented Nov 5, 2017

I don't really understand what you mean. I think I could alternatively just launch the subcommand - this sounds like a real-life example.

Anyway, this can't be merged until #1093 is resolved.

@kbknapp
Copy link
Member

kbknapp commented Nov 6, 2017

@marmistrz sorry, I realize that wasn't super clear of me. I mean how to send all the options and such to the external subcommand, like in this example

@kbknapp kbknapp closed this Feb 23, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants