Conform l20n.js to eslint rules

RESOLVED FIXED

Status

RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: gandalf, Assigned: gandalf)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

3 years ago
Created attachment 8703501 [details] [review]
pull request

Ok, I think it's ready for review. I'll leave some comments in the PR to help you with review.
Assignee: nobody → gandalf
Status: NEW → ASSIGNED
Attachment #8703501 - Flags: review?(stas)
Comment on attachment 8703501 [details] [review]
pull request

Great stuff, thanks for getting this done now before we start working on other patches.  r=me with a nit about XHR's 0 status which I think we still need for file:// URLs to work.
Attachment #8703501 - Flags: review?(stas) → review+
(Assignee)

Comment 3

3 years ago
Commit: https://github.com/l20n/l20n.js/commit/bcf3950875223338c2862de881102f8038aea5dc
Status: ASSIGNED → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.