1579452 - Enable ESLint rule no-async-promise-executor on disabled locations for devtools

Status: RESOLVED FIXED

(DevTools :: General, task, P3)

Description

[Mark Banner (:standard8)]

When we upgraded to ESLint 6, we disabled the no-async-promise-executor rule for various files as it wasn't a quick enable.

This bug is for devtools/

Some notes about this rule (from ESlint's site):

The Promise executor function can be an async function. However, this is usually a mistake, for a few reasons:

• If an async executor function throws an error, the error will be lost and won’t cause the newly-constructed Promise to reject. This could make it difficult to debug and handle some errors.
• If a Promise executor function is using await, this is usually a sign that it is not actually necessary to use the new Promise constructor, or the scope of the new Promise constructor can be reduced.

Comment 1

[Mark Banner (:standard8)]

Attachment: Bug 1579452 - Enable ESLint rule no-async-promise-executor on disabled locations for devtools. r?jlast

Depends on D45010

Comment 2

[Mark Banner (:standard8)]

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=1009ca9870c5663c1103e622f17066d54b829e8e

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/44d3b8582d36
Enable ESLint rule no-async-promise-executor on disabled locations for devtools. r=jlast

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/44d3b8582d36