Closed Bug 1239354 Opened 5 years ago Closed 5 years ago

test_connection_asyncClose.js fails lint checks because of deprecated generator function usage.

Categories

(Toolkit :: Storage, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: mossop, Assigned: mossop)

References

Details

Attachments

(1 file)

No description provided.
Assignee: nobody → dtownsend
Attachment #8707498 - Flags: review?(bugmail) → review+
Comment on attachment 8707498 [details]
MozReview Request: Bug 1239354: Replace old-style generator function with star functions. r?asuth

https://reviewboard.mozilla.org/r/30733/#review27573
I'm very sorry about that; the eslint plugin in atom was throwing all kinds of errors because I use global eslint because of CSP reasons when I invoke atom against only a subtree of gaia.  I tried to globally install the mozilla plugin that was missing, but it's not published on npm, and mach didn't seem to have any magic available.
(In reply to Andrew Sutherland [:asuth] from comment #4)
> I'm very sorry about that; the eslint plugin in atom was throwing all kinds
> of errors because I use global eslint because of CSP reasons when I invoke
> atom against only a subtree of gaia.  I tried to globally install the
> mozilla plugin that was missing, but it's not published on npm, and mach
> didn't seem to have any magic available.

mach eslint --setup will install the right plugins globally including the mozilla plugin.
(In reply to Dave Townsend [:mossop] from comment #5)
> mach eslint --setup will install the right plugins globally including the
> mozilla plugin.

Ahhhh!  Thank you!  I experienced argument-blindness for "./mach eslint --help" and missed it, and so just ran "./mach bootstrap" and "./mach eslint" each in the hope they'd do things.  (Count me as a vote for having "./mach eslint" prompt/suggest the setup automatically like how "./mach build" will aggressively intercede in certain cases.)
(In reply to Andrew Sutherland [:asuth] from comment #7)
> (In reply to Dave Townsend [:mossop] from comment #5)
> > mach eslint --setup will install the right plugins globally including the
> > mozilla plugin.
> 
> Ahhhh!  Thank you!  I experienced argument-blindness for "./mach eslint
> --help" and missed it, and so just ran "./mach bootstrap" and "./mach
> eslint" each in the hope they'd do things.  (Count me as a vote for having
> "./mach eslint" prompt/suggest the setup automatically like how "./mach
> build" will aggressively intercede in certain cases.)

Yeah I want to do that in bug 1239139 when I get chance.
https://hg.mozilla.org/mozilla-central/rev/d2200adcf35b
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.