Turn on experimentalObjectRestSpread in ESlint options

RESOLVED FIXED in Firefox 59

Status

defect
P2
normal
RESOLVED FIXED
2 years ago
a year ago

People

(Reporter: ntim, Assigned: standard8)

Tracking

unspecified
mozilla59
Dependency tree / graph

Firefox Tracking Flags

(firefox59 fixed)

Details

Attachments

(1 attachment)

Reporter

Description

2 years ago
http://eslint.org/docs/user-guide/configuring#specifying-parser-options

Firefox just implemented es object spread (bug 1339395), it would be nice to be able to use it in the source code. Unfortunately, the current eslint config prevents from using object spread. experimentalObjectRestSpread needs to be turned on to use object spread.
Reporter

Comment 1

2 years ago
What do you think ?
Flags: needinfo?(standard8)
Assignee

Comment 2

2 years ago
I think this should be fine. Generally in the past I've seen it requested that new js features aren't used for a release or two. However, seeing as that was implemented in 55 that is probably fine now.
Flags: needinfo?(standard8)
Component: General → Lint
Product: Firefox → Testing
Assignee

Comment 3

2 years ago
We should do this asap after the eslint 4 patches land.
Priority: -- → P2
Comment hidden (mozreview-request)

Comment 5

2 years ago
mozreview-review
Comment on attachment 8928154 [details]
Bug 1375689 - Turn on experimentalObjectRestSpread in ESlint options.

https://reviewboard.mozilla.org/r/199400/#review204594
Attachment #8928154 - Flags: review?(dtownsend) → review+
Assignee

Updated

2 years ago
Assignee: nobody → standard8
Status: NEW → ASSIGNED

Comment 6

2 years ago
Pushed by mbanner@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0b135d7d8cef
Turn on experimentalObjectRestSpread in ESlint options. r=mossop

Comment 7

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/0b135d7d8cef
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59

Updated

a year ago
Product: Testing → Firefox Build System
You need to log in before you can comment on or make changes to this bug.