Require dangling commas via eslint for the migration component

RESOLVED FIXED in Firefox 60

Status

()

enhancement
P4
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: MattN, Assigned: samreadingthompsondevelopment, Mentored)

Tracking

Trunk
Firefox 60
Points:
---

Firefox Tracking Flags

(firefox60 fixed)

Details

()

Attachments

(1 attachment)

In browser/components/migration/.eslintrc.js use:
> "comma-dangle": ["error", "always-multiline"],

Run
> ./mach lint --fix browser/components/migration/

Run
> ./mach test browser/components/migration/

Ensure there are no failures.
Assignee

Comment 1

a year ago
Posted patch 1432062.patchSplinter Review
Attachment #8944570 - Flags: review?(MattN+bmo)
Attachment #8944570 - Flags: review?(MattN+bmo) → review+
Assignee: MattN+bmo → samreadingthompsondevelopment

Comment 2

a year ago
Pushed by ryanvm@gmail.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/be32299523e6
Require dangling commas via eslint for the migration component. r=MattN
Keywords: checkin-needed

Comment 3

a year ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/be32299523e6
Status: ASSIGNED → 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.