Warn on the use of now-deprecated expression closures

RESOLVED WONTFIX

Status

addons.mozilla.org Graveyard
Add-on Validation
RESOLVED WONTFIX
4 years ago
2 years ago

People

(Reporter: kmag, Unassigned)

Tracking

Details

(Whiteboard: [ReviewTeam:P1])

(Reporter)

Description

4 years ago
Expression closures have been deprecated in favor of ES6 arrow functions, and will probably be removed.

We should warn when they're encountered.

Expression closure examples:

let x = function () foo;
let y = function foo(bar) baz;
function z() quux;
(Reporter)

Updated

4 years ago
Whiteboard: [ReviewTeam] → [ReviewTeam:P2]
(Reporter)

Updated

3 years ago
Whiteboard: [ReviewTeam:P2] → [ReviewTeam:P1]
(Reporter)

Updated

3 years ago
Assignee: nobody → kmaglione+bmo
(Assignee)

Updated

3 years ago
Product: addons.mozilla.org → addons.mozilla.org Graveyard
(Reporter)

Updated

2 years ago
Assignee: kmaglione+bmo → nobody
what's the status of this?
(Reporter)

Comment 2

2 years ago
This is a graveyard component. No bugs in it are being worked on. Validator bugs are now tracked on Github.
this repository, right?
https://github.com/mozilla/amo-validator

does it show warning on expression closure now?
if so, it would be nice to close this bug
(or maybe remove bug 995610 from blocking)
(Reporter)

Comment 4

2 years ago
(In reply to Tooru Fujisawa [:arai] from comment #3)
> this repository, right?
> https://github.com/mozilla/amo-validator

Yes.

> does it show warning on expression closure now?

I don't know. I haven't been involved with the validator for the past year.
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.