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

add an example with clapv3 (beta) with optional stdin/stdout #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

add an example with clapv3 (beta) with optional stdin/stdout #37

wants to merge 1 commit into from

Conversation

michelk
Copy link

@michelk michelk commented Jun 11, 2021

This addressed #36

@bbros-dev
Copy link

bbros-dev commented Jul 12, 2021

This is interesting thank you making the effort.
Does this mean the app could work with piped input/output?

yes | cli-args-clap3-stdin-stdout | xargs echo

@bbros-dev
Copy link

Hmm, it would appear these two issues are relevant:

Hope that helps some one?

@epage
Copy link
Contributor

epage commented Jul 19, 2021

Thanks for wanting to improve the book!

Some quick thoughts

  • It seems like we should be consistent in clap3 vs structopt. Personally, I'd wait for clap3 to be released and then completely switch over
  • This seems like this belongs more in the in-depth section than the tutorial.

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