1878009 - Remove Babel integration with ESLint

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, task, P2)

Description

[Mark Banner (:standard8)]

We originally had Babel integrated with ESLint due to wanting to be able to use some language features that were in development - they were ECMAScript stage 3 proposals, but had not got to stage 4. ESlint doesn't support new features until stage 4, so we used the Babel integration to work around that.

That has long since passed, and we have not had more cases of needing to use Babel in the interim. Whilst this may change in future, removing Babel gives a decent performance increase (1m 56 -> 1m 32 on my Mac M1 Pro). So for now, we've decided to remove it, we can always refer back and add it back in later if the need arises.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1878009 - Remove Babel integration from ESLint. r?Mossop!

Comment 2

[Mark Banner (:standard8)]

Attachment: Bug 1878009 - Remove Babel from ESLint's node-modules. r?Mossop!

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/969ecb7fe02e
Remove Babel integration from ESLint. r=mossop,devtools-reviewers,nchevobbe
https://hg.mozilla.org/integration/autoland/rev/4412c4c8dbf0
Remove Babel from ESLint's node-modules. r=mossop

Comment 4

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/969ecb7fe02e
https://hg.mozilla.org/mozilla-central/rev/4412c4c8dbf0