V2.29.3 #1164
Merged
V2.29.3 #1164
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
…elves This allows properly supporting shell aliases and config files. Before, if an option only accepted a single value and that option was set inside an alias, the CLI could never override that value. For instance: ``` $ alias prog='prog --option=value' $ prog --option=other ``` This would cause an error. Now clap gracefully accepts the new value to replace the old value. Closes #976
… all args to override themselves Relates to #976
…show up as required Prior to this commit if one had two requirements which conflicted with each other, and an error was generated due to a totally seperate argument, BOTH required but conflicting args would show up in the help message as "missing but required" This is now fixed and the *used* required arg shows up, but the conflicting required arg has been dropped. Closes #1158
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.
Fixes:
cc @BurntSushi this should allow true self-overrides for args, and a convenience setting of
AppSettings::AllArgsOverrideSelfif you want all args to be able to override themselves (minus positional arguments)This change is