Description
I don't think no-unused-vars rule should check this case, because the variable is actually "used" in some case, only should not assign when it's a last use.
Fail
function foo(bar) {
bar = doSomething();
}function foo(bar) {
bar = doSomething(bar);
}function foo(bar) {
bar = doSomething(bar);
// ... do something else, but not using bar
}Pass
function foo(bar) {
bar = doSomething(bar);
return bar;
}function foo(bar) {
bar = doSomething(bar);
use(bar);
}function foo(bar) {
function aInnerFunctionUsesBar() {
return bar;
}
bar = doSomething(bar);
// bar is not used but calling another function uses bar
aInnerFunctionUsesBar()
}
sindresorhus / eslint-plugin-unicorn
More than 100 powerful ESLint rules
4.2k
How does funding with Polar work?
1
Pay now to fund the work behind this issue.
2
Get updates on progress being made.
3
Maintainer is rewarded once the issue is completed.
FAQ
Backer
You're funding impactful open source efforts
Contributor
You want to contribute to this effort
Maintainer
You want to get funding like this too