Skip to content
This repository has been archived by the owner on Mar 23, 2024. It is now read-only.

Allow false (and/or perhaps undefined) values for boolean options #287

Closed
cvrebert opened this issue Mar 6, 2014 · 3 comments
Closed

Allow false (and/or perhaps undefined) values for boolean options #287

cvrebert opened this issue Mar 6, 2014 · 3 comments

Comments

@cvrebert
Copy link
Contributor

cvrebert commented Mar 6, 2014

Use case: When merging multiple sets of options (e.g. per-file overrides) and you need to relax an option. Currently, using false or undefined in an overriding set of options results in errors, e.g.:

requireCamelCaseOrUpperCaseIdentifiers option requires true value or should be removed
requireCamelCaseOrUpperCaseIdentifiers option requires boolean value

(This is an intentional duplicate of #255, which was prematurely closed.)

cvrebert added a commit to twbs/bootstrap that referenced this issue Mar 6, 2014
@mikesherov
Copy link
Contributor

@cvrebert can you use null? We reserved false and true for future configuration changes. Does null work?

@cvrebert
Copy link
Contributor Author

cvrebert commented Mar 6, 2014

Yes, it does! Hadn't thought to try it. Would be nice if that was documented more clearly.
Thanks!

@cvrebert cvrebert closed this as completed Mar 6, 2014
@mikesherov
Copy link
Contributor

@cvrebert yeah, we need to do a better job documenting and making options easier to configure. It's the next big thing we're working on. Thanks for bearing with us!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants