Closed
Bug 1639814
Opened 4 years ago
Closed 4 years ago
Port bug 1617959: Switch to the babel-eslint parser
Categories
(Thunderbird :: General, task)
Thunderbird
General
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 78.0
People
(Reporter: darktrojan, Assigned: darktrojan)
References
Details
Attachments
(1 file)
1.35 KB,
patch
|
mkmelin
:
review+
|
Details | Diff | Splinter Review |
Firefox switched to the babel parser for (among other things) the ability to parse the ?.
operator. We should follow suit, especially as I'm about to use one of those operators in a file ported from Firefox.
Assignee | ||
Comment 1•4 years ago
|
||
This is the simplest fix. Rather than a separate babel config file I've inlined the config, as we don't have the same reasons for using a separate file.
It also stops using the html plugin. The Firefox config doesn't have this but we do. I don't know why.
Attachment #9150691 -
Flags: review?(mkmelin+mozilla)
Assignee | ||
Comment 2•4 years ago
|
||
Okay, I was wrong, we're not stopping using the html plugin, and Firefox do use it. It's added elsewhere (recommended.js).
Updated•4 years ago
|
Attachment #9150691 -
Flags: review?(mkmelin+mozilla) → review+
Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/a15cee0df19f
Port bug 1617959: Switch to the babel-eslint parser. r=mkmelin
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•4 years ago
|
Target Milestone: --- → Thunderbird 78.0
You need to log in
before you can comment on or make changes to this bug.
Description
•