Align flag alias with normal use #85
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The use a single '-' is usually for short options a single char. Also the way it is used in all tests. One of the example has an alias "ag" which currently is parsed if the argument is '-ag' with this change aliases with more the a single char need to be parsed with double dash - as e.g. '--ag'
This is also the normal behavior of CLI arg parsers - you can group mutilple short options with a single dash.
jhilmer
force-pushed
the
align-flag-alias-with-normal-use
branch
from
7f4e14e to
ea29a49
Compare
|
@jhilmer |
jhilmer
force-pushed
the
align-flag-alias-with-normal-use
branch
from
9ba3222 to
e88ffef
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Align use of '-' and '--' with other CLI parsers
The use a single '-' is usually for short options a single char. Also
the way it is used in all tests. One of the example has an alias "ag"
which currently is parsed if the argument is '-ag' with this change
aliases with more the a single char need to be parsed with double
dash - as e.g. '--ag'