1889350 - Broken import in eslint-plugin-mozilla's chrome-worker.js

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, defect)

Description

[Shane Hughes [:aminomancer]]

This line should be changed to fix the import. This isn't supposed to trigger unless the chrome-worker environment is used, but for some mysterious reason I'm encountering it even though I only use browser-window. Still, it's worth fixing as long as the code exists.

diff --git a/tools/lint/eslint/eslint-plugin-mozilla/lib/environments/chrome-worker.js b/tools/lint/eslint/eslint-plugin-mozilla/lib/environments/chrome-worker.js
index db5759b26c3ec..a00b55401bdcc 100644
--- a/tools/lint/eslint/eslint-plugin-mozilla/lib/environments/chrome-worker.js
+++ b/tools/lint/eslint/eslint-plugin-mozilla/lib/environments/chrome-worker.js
@@ -10,7 +10,7 @@
 
 "use strict";
 
-var globals = require("globals");
+var globals = require("../globals");
 var util = require("util");
 
 var workerGlobals = util._extend(

Comment 1

[Shane Hughes [:aminomancer]]

Attachment: Bug 1889350 - Remove chrome-worker environment from eslint-plugin-mozilla. r=Standard8

Comment 2

[BugBot [:suhaib / :marco/ :calixte]]

The severity field is not set for this bug.
:andi, could you have a look please?

For more information, please visit BugBot documentation.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8cdd31c08963
Remove chrome-worker environment from eslint-plugin-mozilla. r=Standard8

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/8cdd31c08963