Add rule to warn about use of `arguments` in fat arrow functions

Please describe what the rule should do:

warn on usage of arguments inside fat arrow functions

What category of rule is this? (place an “X” next to just one item)

[ ] Enforces code style [x] Warns about a potential error [ ] Suggests an alternate way of doing something [ ] Other (please specify:)

Provide 2-3 code examples that this rule will warn about:

const logArgs = () => console.log(arguments);

function foo(a, b, c) {
  const bar = blah => {
    if(arguments.length > 1) {
      console.log("too many args!");
    }
  };
  bar(123);
}

foo(456, 789);

Why should this rule be included in ESLint (instead of a plugin)?

arguments is not bound for fat arrow functions. The first example is quite easy to catch by a developer, as an error will be thrown. The second example would be hard to track down because arguments is actually referring to foo’s arguments, not bar’s

[WickyNilliams]

or even in ES5:

function getFetchFunction() {
    var args = arguments;
    return () => fetch.apply(null, args);
}

Both of these are much clearer

[WickyNilliams]

Now I’ve thought of it more, I think you’re right. May as well use rest params in every case. I didn’t spot the prefer-rest-params rule initially. Closing