1330055 - (dt-eslint-rules) Enable more eslint rules for devtools/

Status: NEW

(DevTools :: General, enhancement, P3)

Description

[Tim Nguyen :ntim]

https://msujaws.wordpress.com/2017/01/10/eslint-updates-for-firefox-developers/

Not in .eslintrc:
    no-extra-label
    no-self-assign
    no-unused-labels
    no-useless-call
    no-useless-return

Disabled in .eslintrc:
    no-iterator
    object-shorthand
    no-regex-spaces <- We've just disabled that one since it's not very useful for 1-4 spaces

Already enabled:
    brace-style
    no-multi-spaces
    no-debugger
    no-delete-var
    no-sparse-arrays
    no-unsafe-finally
    no-cond-assign
    no-extra-bind
    no-unsafe-negation
    no-lone-blocks

Comment 1

[Tom Tromey :tromey]

I was thinking it might be nice to have a "complete" eslintrc, where every option is
mentioned, even if just to enable the default, so that we can comment on each decision,
and then have a record so that when there's a new release we can see what might need
to be added.