v3: disallowUnusedVariables rule warns on unused function names.

jscs-dev / node-jscs

:arrow_heading_up: JavaScript Code Style checker (unmaintained)

https://jscs-dev.github.io

MIT License

4.97k stars 515 forks source link

Closed ljharb closed 8 years ago

ljharb commented 8 years ago

disallowUnusedVariables: Variable `Bar` is not used at test.js :
    68 |  var Foo = function Bar() {};
-------------------------------^

Function names are not variables, despite being lexical bindings, and this rule should not be warning on them.

mdevils commented 8 years ago

Thanks, will fix it soon.

mdevils commented 8 years ago

Fixed in 3.0.1. Please check it out.

ljharb commented 8 years ago

Thanks!

ljharb commented 8 years ago

Confirmed fixed