[BUG] Add "max-len": ["error", 80] to .eslintrc.json and make it work

[isaacs]

We have a lot of long lines, let's shorten them.

Be smart about making the choices to split lines, no fake internet points for cutting strings and making them unreadable!

[ljharb]

You can configure max-len to ignore lines with long strings or regexes or urls; i suggest doing so. Example: https://github.com/airbnb/javascript/blob/2c5c88d0487db3024e8a20bb20f46933a03dd8aa/packages/eslint-config-airbnb-base/rules/style.js#L205

[dominguetigs]

@isaacs
Is anyone doing this issue?

[Gurinderp]

Could I take on this issue if no one is currently working on it? @isaacs
Not sure if @dominguetigs still wants it.

[dominguetigs]

You can do it @Gurinderp

[sainple0001]

Can you assign me on this issue ? @isaacs

[Edu93Jer]

Hi @isaacs is anyone still working on the case? If no, could you please explain to me a little more about the project, I'm a newbie and would like to contribute, or is the issue is closed?

[Edu93Jer]

@darcyclarke - @isaacs, so I just added the "max-len" to the .eslintrc.json, but what do you mean with make it work?
It should be working right, just adding this rules...
"max-len": ["error", 80, { "ignoreUrls": true, "ignoreComments": false, "ignoreRegExpLiterals": true, "ignoreStrings": true, "ignoreTemplateLiterals": true }]

[isaacs]

Well, there are quite a few places (iirc?) that are longer than 80 chars. So npm run lint should not raise any errors with that option.

[a2br]

Looks like #2361 has been merged. Shouldn't this issue be closed too?