Closed
Bug 1917003
Opened 2 months ago
Closed 1 month ago
Prevent using await await in JS
Categories
(Developer Infrastructure :: Lint and Formatting, task, P3)
Developer Infrastructure
Lint and Formatting
Tracking
(firefox132 fixed)
RESOLVED
FIXED
132 Branch
Tracking | Status | |
---|---|---|
firefox132 | --- | fixed |
People
(Reporter: jdescottes, Assigned: jdescottes)
References
Details
(Whiteboard: [webdriver:m12])
Attachments
(2 files)
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.
Assignee | ||
Comment 1•2 months ago
|
||
Updated•2 months ago
|
Assignee: nobody → jdescottes
Status: NEW → ASSIGNED
Assignee | ||
Comment 2•2 months ago
|
||
Depends on D221187
Pushed by jdescottes@mozilla.com:
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•1 month ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b45070bc3aaa
https://hg.mozilla.org/mozilla-central/rev/d2267c0307e5
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
status-firefox132:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 132 Branch
Assignee | ||
Updated•1 month ago
|
Priority: -- → P3
Whiteboard: [webdriver:m12]
You need to log in
before you can comment on or make changes to this bug.
Description
•