Closed Bug 1843797 Opened 1 year ago Closed 1 year ago

Add ESLint rule to reject ChromeUtils.import

Categories

(Developer Infrastructure :: Lint and Formatting, task)

Product:
Developer Infrastructure
Component:
Lint and Formatting
Type:
task

Tracking

(firefox118 fixed)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
firefox118 --- fixed

People

(Reporter: arai, Assigned: arai)

References

Details

Attachments

(1 file)

Bug 1843797 - Add ESLint rule to reject ChromeUtils.import. r?Standard8!
48 bytes, text/x-phabricator-request
Details | Review

(In reply to Mark Banner (:standard8) (afk until 24th July) from bug 1835030 comment #0)

As esm-ification is getting nearer to completion, we'll need to additionally update imports in .html/.xhtml files.

They look fairly simple, so we might be able to use a ESLint rule on a lot of them to do the initial fixing, and then maybe land just the rule (without the auto-fix) to stop any new ones being added.

Attachment #9344109 - Attachment description: WIP: Bug 1843797 - Add ESLint rule to reject ChromeUtils.import. → Bug 1843797 - Add ESLint rule to reject ChromeUtils.import.
Attachment #9344109 - Attachment description: Bug 1843797 - Add ESLint rule to reject ChromeUtils.import. → WIP: Bug 1843797 - Add ESLint rule to reject ChromeUtils.import.
Attachment #9344109 - Attachment description: WIP: Bug 1843797 - Add ESLint rule to reject ChromeUtils.import. → Bug 1843797 - Add ESLint rule to reject ChromeUtils.import. r?Standard8!
Pushed by arai_a@mac.com: https://hg.mozilla.org/integration/autoland/rev/9f0c33658467 Add ESLint rule to reject ChromeUtils.import. r=Standard8

https://hg.mozilla.org/mozilla-central/rev/9f0c33658467

Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox118: --- → fixed
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: