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

Allow false values #255

Closed
BryanDonovan opened this issue Feb 20, 2014 · 1 comment
Closed

Allow false values #255

BryanDonovan opened this issue Feb 20, 2014 · 1 comment

Comments

@BryanDonovan
Copy link
Contributor

It seems to me that it would be very useful to allow both boolean values for all boolean rules. E.g., disallowMultipleLineBreaks can be set to true but not false.

AssertionError: disallowMultipleLineBreaks option requires true value or should be removed

It's nice to be able to set rules like this to either value when trying out the features, and so other devs can see what rules are available without having to look up the node-jscs docs. It wouldn't be so bad if JSON allowed use to comment-out lines.

@BryanDonovan
Copy link
Contributor Author

Nevermind this.. I can see the reason for not allowing more than one value per rule.

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

1 participant