-
Notifications
You must be signed in to change notification settings - Fork 76
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
version, --version, -V, -v weirdness #795
Comments
-v is short of --verbose. I think we should get rid of -V. But I think it is strange that --version differs from the subcommand. However, I think is is difficult because the flag is autogenerate by clap and the other requires build time info. |
Ideally, we would replace |
This is a common pattern in CLIs. For example, I think this issue is pretty low impact. But for the sake of having one way to do version in soroban-cli, I suggest we remove The change required is minimal: |
-V
vs-v
seems like an anti-pattern. If we want to shorten the--verbose
command it should be-V
and-v
should be version and it should always print out all the version not just thesoroban
cli version.The text was updated successfully, but these errors were encountered: