Closed Bug 1702539 Opened 2 years ago Closed 2 years ago

Ensure browser/extensions/screenshots/background/selectorLoader.js passes eslint check

Categories

(Firefox :: Screenshots, enhancement, P3)

Product:
Firefox
Component:
Screenshots
Type:
enhancement

Tracking

()

Status:
RESOLVED FIXED
Milestone:
89 Branch
Tracking Flags:
Tracking Status
firefox89 --- fixed

People

(Reporter: emmamalysz, Assigned: kajalsah107)

References

Details

(Whiteboard: [outreachy-2021-screenshots])

Attachments

(1 file)

Bug 1702539 - Fixes eslint errors in browser/extensions/screenshots/background/selectorLoader.js. r?emalysz
48 bytes, text/x-phabricator-request
Details | Review

+++ This bug was initially created as a clone of Bug #1702532 +++

We should fix these errors in order to enable eslint in the screenshots directory.

To find the errors:

  1. Temporarily comment out https://searchfox.org/mozilla-central/rev/4fa18c26fa907f38d56b599571b9846af1506f3c/.eslintignore#43-45
  2. Run ./mach lint browser/extensions/screenshots/background/selectorLoader.js

First, check if this can receive an automatic fix via:
./mach lint browser/extensions/screenshots/background/selectorLoader.js --fix.

Otherwise, investigate the failures and see if it requires a manual fix

Severity: -- → S4
Priority: -- → P3
Whiteboard: [outreachy-2021-screenshots]
Assignee: nobody → kajalsah107
Status: NEW → ASSIGNED
Pushed by emalysz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/65dc61a78542
Fixes eslint errors in browser/extensions/screenshots/background/selectorLoader.js. r=emalysz

https://hg.mozilla.org/mozilla-central/rev/65dc61a78542

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
status-firefox89: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
You need to log in before you can comment on or make changes to this bug.