1917003 - Prevent using await await in JS

Status: RESOLVED FIXED

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

Description

[Julian Descottes [:jdescottes]]

While using await await somePromise; is a valid syntax, it's almost always a mistake.
For the rare cases where this would be useful it sounds fine to require a comment + expect developers to disable the eslint rule locally.

Comment 1

[Julian Descottes [:jdescottes]]

Attachment: Bug 1917003 - [eslint] Add linter rule to ban await await

Comment 2

[Julian Descottes [:jdescottes]]

Attachment: Bug 1917003 - [eslint] Fix violations for reject-multiple-await

Depends on D221187

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b45070bc3aaa
[eslint] Add linter rule to ban await await r=Standard8,frontend-codestyle-reviewers
https://hg.mozilla.org/integration/autoland/rev/d2267c0307e5
[eslint] Fix violations for reject-multiple-await r=perftest-reviewers,credential-management-reviewers,places-reviewers,issammani,daisuke,sparky

Comment 4

[Norisz Fay [:noriszfay]]

https://hg.mozilla.org/mozilla-central/rev/b45070bc3aaa
https://hg.mozilla.org/mozilla-central/rev/d2267c0307e5