proposal(eslintrc): warn for arrow-parens rule

[Scarysize]

Does anyone remember why we set this to 0? Please discuss.

[mjaros]

If we set this to 1 without any configuration it should enforce braces, right? Maybe what we want is "as-needed": http://eslint.org/docs/rules/arrow-parens

[Scarysize]

That sounds good.

[pwambach]

"as-needed": 1 👍 so not for a => {} but for (a, b) => {}

[mrMetalWood]

Plus one for as-needed 👍

[usefulthink]

question, how would eslint know what I want to do here?

function a(arg, cb) {}
function b(...args) {}
function c(arg) {}

a(something, cbArg => callback(cbArg));
b(something, cbArg => callback(cbArg));
c((cbArg1, cbArg2) => cbArg1 + cbArg2);

[aemkei]

IMO as-needed is what we want here.

http://eslint.org/docs/rules/arrow-parens#options
http://eslint.org/docs/rules/arrow-parens#as-needed

@usefulthink Guess ESLint is not checking the parameters but only the arguments with this rule.