Allow `no-shadow` rule to not check arguments

It often seems better to allow no-shadow rule to not check argument variables.

Arguments are often need only inside function’s own context, not deeper, e.g. users can use nesting .map(function(v, k){}) with “common” argument names v, k if result of parent function is not used inside of child.

no-shadow can give unneccessary error if user wants to make scope with .bind.

for (var i = 0; i <= 10; i++) {
    funcs.push(function(i){ // err here
    }.bind(null, i));
};

It slightly conflicts with handle-callback-err and no-unused-vars with {args: 'after-used'}

// eslint handle-callback-err:0
doSomething(function(err, res) {
   doAnother(res, function(err, res2) { // wish not have error here, but should add no-shadow: 0 and probably suppress real bugs

  ));
});

I suggest to add option to no-shadow rule to not check arguments. It should work something like this.

/*eslint no-shadow:[1, "skip-args"]*/
function some(foo, baz) {
   var bar = 1;
    return function(foo, bar) { // no err here
       var baz = 42; // err here
   };
}

<bountysource-plugin>

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource. </bountysource-plugin>

[ilyavolodin]

I don’t think it’s the same issue as #2645 at all. That one is talking about making an exception for function names, this one is talking about function arguments. I definitely agree with the need for #2645 I’m not sold on this one though.

[alberto]

Closing due to lack of activity