Skip to content
This repository has been archived by the owner on Dec 29, 2020. It is now read-only.

disallowMultipleVarDecl inversion #29

Closed
Baptouuuu opened this issue Jan 19, 2014 · 1 comment
Closed

disallowMultipleVarDecl inversion #29

Baptouuuu opened this issue Jan 19, 2014 · 1 comment

Comments

@Baptouuuu
Copy link

In my config I use disallowMultipleVarDecl: true but I run the task it reports that my code is not matching rule on code like that:

var foo = 'bar',
    bar = 'baz';

for (var i = 0, l = 42; i < l; i++) {
    // doing stuff
}

Is it me not understanding the meaning of disallowMultipleVarDecl keyword?
But I really think it's inverted with requireMultipleVarDecl, as when I set the last one to true it passes.

@markelog
Copy link
Member

It's not a jscs tracker – https://github.com/mdevils/node-jscs.

With code above, disallowMultipleVarDecl: true, should report a error, but if you set requireMultipleVarDecl: true without disallowMultipleVarDecl and it passes and it is a bug.

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