1787278 - Some newtab instrumentation module(s) don't parse ES2020 constructs

Status: RESOLVED FIXED

(Firefox :: Messaging System, defect, P3)

Description

[Shane Hughes [:aminomancer]]

It would be good to update the jsm/jsx compilation and unit tests to handle more modern ES syntax. We are currently using babel plugins to parse and transpile nullish coalescing and optional chaining operators, but they make the output far less readable. We could remove them, but for istanbul-instrumenter-loader.

According to this report, we may be able to use babel-plugin-istanbul instead.

Comment 1

[Punam Dahiya [:pdahiya]]

NI @dmosedale to help link this under a Meta bug tracking infrastructure fixes.

Comment 2

[Dan Mosedale (:dmosedale, :dmose)]

:emcminn replaced istanbul-instrumenter-loader with coverage-istanbul-loader in bug 1825144. I wonder if this is now unblocked...

Clearing the needinfo, as this is already blocking fx-ms-build.

Comment 3

[Mark Banner (:standard8)]

Attachment: Bug 1787278 - Drop babel transpile plugins from newtab and related code. r?aminomancer!

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/37abb574b134
Drop babel transpile plugins from newtab and related code. r=aminomancer

Comment 5

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/37abb574b134