-
Notifications
You must be signed in to change notification settings - Fork 355
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 a few tests related to github issues #666
Conversation
ec4fb28
to
1603ccb
Compare
Try to address #669 |
Closes #670 |
I don't like embedding an English label like this into CLI11; it's nice to avoid text that might need to be translated. Argparse uses "[XXXX]", that might be clearer that the current form? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would you mind summarizing the changes for the changlog? Changing the default help print, a fix related to RTTI, and a few extra tests?
…X, this was confusing in some cases particularly with flags or with multiple option names. Now is [default=XXXX] which makes it clearer what the value represents.
add test related to issue #556, #569, #551
Changes the way default_str is displayed in the help messages from "=XXXX" to "[XXXX]" to make it clearer what the purpose of the value is.