Closed
Bug 1889087
Opened 2 years ago
Closed 2 years ago
Enable ESLint rule mozilla/reject-chromeutils-import on devtools as well
Categories
(Developer Infrastructure :: Lint and Formatting, task, P3)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox126 fixed)
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| firefox126 | --- | fixed |
People
(Reporter: standard8, Assigned: standard8)
References
Details
Attachments
(1 file)
The ESLint rule mozilla/reject-chromeutils-import can now also be enabled on devtools since the conversion from jsm to ESM is complete there.
There's one instance where there is code that could call it, but that code should be unused now.
|
Assignee | |
Comment 1•2 years ago
|
||
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/50f86f169ee8
Enable ESLint rule mozilla/reject-chromeutils-import on devtools as well. r=arai,devtools-reviewers,profiler-reviewers,julienw
|
||
Comment 3•2 years ago
•
|
||
Backed out for causing talos performance test failures
Backout link: https://hg.mozilla.org/integration/autoland/rev/c720e2e99bf375c24b09c0a1415f15ba911c56ce
Flags: needinfo?(standard8)
|
|
Assignee | |
Comment 4•2 years ago
|
||
The damp tests need conversion from .jsm to .sys.mjs as well, I'm working on a fix.
Flags: needinfo?(standard8)
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/1e56e647f8d5
Enable ESLint rule mozilla/reject-chromeutils-import on devtools as well. r=arai,devtools-reviewers,profiler-reviewers,julienw,frontend-codestyle-reviewers,perftest-reviewers,sparky
|
||
Comment 6•2 years ago
|
||
| bugherder | ||
You need to log in
before you can comment on or make changes to this bug.
Description
•