-
Notifications
You must be signed in to change notification settings - Fork 65
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
Show valid options in help where possible #88
Comments
@notmandatory I'm taking a look at this one, it's still up for takers? |
@oleonardolima you can try to apply them on top of #104 if you are still up for it. That will be the final version of the crate after the big changes.. So you don't have to make duplicate works.. |
@rajarshimaitra great! thanks for the suggestion, I'll work on top of it. |
@notmandatory @rajarshimaitra I've gone through all the available commands in the current state of #104 as Raj suggested. But I didn't find any other than the @rajarshimaitra as it's on top of #104 I opened a PR to your branch, wdyt ? |
Hi, sorry for the delay in getting to this, it's Ok if only network needs to be changed. You should be able to make your PR against the |
@notmandatory no problem 😁 , actually Raj got the commit 2999a3d with the changes into the branch for #104. |
Anywhere where there are only a limited enum set of options, such as with the network option, we should show in the help what those options are. This is done with the
possible_values
structopt feature, see example in Compile command:The text was updated successfully, but these errors were encountered: