Enable ESLint rule mozilla/use-services for browser/

RESOLVED FIXED in Firefox 60

Status

()

enhancement
P3
normal
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: standard8, Assigned: standard8)

Tracking

unspecified
Firefox 60
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox60 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
As we've been doing elsewhere, this continues the work to replace .getService() with Services.jsm uses where possible and enable the mozilla/use-services rule.
(Assignee)

Updated

2 years ago
Depends on: 1421968
(Assignee)

Updated

2 years ago
Blocks: 1421969
Summary: Enable ESLint rule mozilla/use-services for browser/ → [meta] Enable ESLint rule mozilla/use-services for browser/
(Assignee)

Comment 1

a year ago
Not a meta, since I need to do tidy up work in /.eslintrc.js for the imports that have been fixed elsewhere.
Summary: [meta] Enable ESLint rule mozilla/use-services for browser/ → Enable ESLint rule mozilla/use-services for browser/
Comment hidden (mozreview-request)

Comment 3

a year ago
mozreview-review
Comment on attachment 8947403 [details]
Bug 1421379 - Finish enabling ESLint's mozilla/use-services in browser/.

https://reviewboard.mozilla.org/r/217126/#review223046
Attachment #8947403 - Flags: review?(dtownsend) → review+

Comment 4

a year ago
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/97ac300c64d0
Finish enabling ESLint's mozilla/use-services in browser/. r=mossop

Comment 5

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/97ac300c64d0
Status: NEW → RESOLVED
Last Resolved: a year ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in before you can comment on or make changes to this bug.