Add `max-len` rule ? about eslint-config-xo HOT 5 CLOSED

Ok it works by using :

"xo": {
    "esnext": true,
    "rules": {
      "max-len": ["error", 80, 2] // ["error", max-len, tab-witdh] (i use 2, default is 4)
    }

Thanks for your help ! 👍

from eslint-config-xo.

Editors can soft wrap, but is a line like this considered "linted"?

const element = <div className="AvatarStack AvatarStack--three-plus AvatarStack--right clearfix d-inline-block" style={{marginTop: 1}}><div className="AvatarStack-body tooltipped tooltipped-sw tooltipped-align-right-1" aria-label={usernames}>{avatars}</div></div>;

Perhaps there should still be a sensible limit.

from eslint-config-xo.

Why? Editors today can soft-wrap, so just set the line-length you prefer in your editor.

from eslint-config-xo.

Yes but not anyone is using modern editors, by the way it's an usal convention I always met. Maybe just add the option for configuration in package.json.

from eslint-config-xo.

I'm pretty sure every code editor in existence today can soft-wrap. I'm aware it's a common convention to hard-wrap, but I don't do stuff just because it's a convention. You can already use this rule yourself in your ESLint config or the rules option if you use XO ;)

I'll keep this open for discussion.

from eslint-config-xo.