Change defaults

[niftylettuce]

Don't you need to change https://github.com/meetearnest/eslint-config-earnest-es7/blob/master/index.js#L50 and https://github.com/meetearnest/eslint-config-earnest-es7/blob/master/index.js#L51 to "either" per standard/standard#182?

[niftylettuce]

I meant 0 for off***

[yjhuoh]

[niftylettuce]

haha what

[niftylettuce]

I can close this, I extended it with my own rules, but I thought these were different from standard, not sure, haha

[jsatk]

Thanks for the feedback @niftylettuce. Will look into it.

[bromanko]

I think @niftylettuce's proposal is reasonable and in line with how we treat spaces in array literals. I suggest we change object-curly-spacing to 0.

@moredip, @jsatk what say you?

[jsatk]

@bromanko I agree. Sounds good to me.

[moredip]

Fine with me.
On May 25, 2016 11:07 AM, "Jesse Atkinson" notifications@github.com wrote:

@bromanko https://github.com/bromanko I agree. Sounds good to me.

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#16 (comment)