Skip to content
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.

Sign up for GitHub

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

Jump to bottom

use_delimiter(false) should be the default without multiples #655

Closed
kbknapp opened this issue Sep 10, 2016 · 0 comments
Closed

use_delimiter(false) should be the default without multiples #655

kbknapp opened this issue Sep 10, 2016 · 0 comments
Labels
C-enhancement Category: Raise on the bar on expectations
Milestone
2.12.0

Comments

@kbknapp
Copy link
Member

kbknapp commented Sep 10, 2016

This has been an issue with a few projects. Because use_delimiter(true) is the default, which is meant for multiple values, but it takes affect even with a single value for an option or arg. This means a valid single value, --option "some thing, with commas, and stuff" will get split up and causes confusing errors.

use_delimiter(false) should be default unless multiple(true) or similar has been set (such as number_of_values etc.)
@kbknapp kbknapp added C-enhancement Category: Raise on the bar on expectations P2: need to have labels Sep 10, 2016
@kbknapp kbknapp added this to the 2.12.0 milestone Sep 10, 2016
@kbknapp kbknapp mentioned this issue Sep 10, 2016
Closed
@homu homu closed this as completed in f9e6925 Sep 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-enhancement Category: Raise on the bar on expectations
Projects
None yet
Milestone
2.12.0
Development

No branches or pull requests
1 participant
@kbknapp