1778106 - Move Pocket code to use mjs files, rather than defining sourceType for ESLint

Status: RESOLVED FIXED

(Firefox :: Pocket, task)

Description

[Mark Banner (:standard8)]

With the transition to using ES modules rather than JSMs getting started, we are now using the .mjs extension for those files.

We'd like to transition our content code that is using modules to use the .mjs extension as well - the primary gains of this are that we a) don't need to manually specify sourceType: module for ESLint, b) this gives a visual distinction to developers that the file is a module rather than a regular JavaScript file.

This bug is for the pocket files.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1778106 - Move Pocket code to use mjs files, rather than defining sourceType for ESLint. r?thecount!

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6ce209a21e39
Move Pocket code to use mjs files, rather than defining sourceType for ESLint. r=thecount

Comment 3

[Natalia Csoregi [:nataliaCs]]

https://hg.mozilla.org/mozilla-central/rev/6ce209a21e39

Comment 4

[Dianna Smith [:diannaS]]

:standard8 could this be affecting some Pocket tests on Android CI?

Comment 5

[Mark Banner (:standard8)]

I think it is unlikely, as this code is in browser/ which should be Firefox desktop specific, and I can't see any obvious references to it in the Android repositories.