Bug 1330055 (dt-eslint-rules)

Enable more eslint rules for devtools/

NEW
Unassigned

Status

P3
enhancement
2 years ago
4 months ago

People

(Reporter: ntim, Unassigned)

Tracking

(Depends on: 3 bugs, Blocks: 1 bug)

Firefox Tracking Flags

(Not tracked)

Details

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

2 years ago
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.
Depends on: 1330074
Depends on: 1330099
Depends on: 1311078
Depends on: 1330148
Severity: normal → enhancement
Priority: -- → P3
No longer blocks: 1402832
(Reporter)

Updated

6 months ago
Depends on: 1454696
(Reporter)

Updated

6 months ago
Alias: dt-eslint-rules

Updated

4 months ago
Product: Firefox → DevTools
You need to log in before you can comment on or make changes to this bug.