Closed Bug 1554169 Opened 4 months ago Closed 4 months ago

Enable some ESLint rules for more of netwerk/

Categories

(Core :: Networking, task, P2)

task
Points:
1

Tracking

()

RESOLVED FIXED
mozilla69
Iteration:
69.1 - May 13 - 26
Tracking Status
firefox69 --- fixed

People

(Reporter: standard8, Assigned: standard8)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

As part of rolling out ESLint across the tree, I'd like to enable it for the netwerk/ directory.

The first stage to this is enabling ESLint on the directories, and disabling the rules that fail. This gives us the benefit of at least syntactical checks to begin with plus some rules being enabled.

Some of the disabled rules are style related, those we'll defer to later as we're looking into rolling out js formatting across the tree (and might as well take the reformatting hit one-time).

For enabling the other rules I will file mentored follow-up bug(s) after this lands.

Priority: -- → P2
Whiteboard: [necko-triaged]
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/10822a2ce5ae
Enable some ESLint rules for more of netwerk/. r=dragana
Status: NEW → RESOLVED
Closed: 4 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla69
Blocks: 1556844
Iteration: --- → 69.1 - May 13 - 26
You need to log in before you can comment on or make changes to this bug.