Closed
Bug 1156089
Opened 10 years ago
Closed 10 years ago
Fix remaining fl jshint issues
Categories
(Firefox OS Graveyard :: Gaia::Gallery, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: kgrandon, Assigned: kgrandon)
References
Details
Attachments
(1 file)
We're getting close to being done with jshint fixes and can kill off some legacy code once we do so. Let's push this through!
Comment 1•10 years ago
|
||
Assignee | ||
Comment 2•10 years ago
|
||
Comment on attachment 8594524 [details] [review]
[gaia] KevinGrandon:bug_1156089_fl_jshint > mozilla-b2g:master
David - Can you take a look at or redirect this when you get a chance? Just fixing the last remaining jshint issues here. Thanks!
Attachment #8594524 -
Flags: review?(dflanagan)
Comment 3•10 years ago
|
||
Comment on attachment 8594524 [details] [review]
[gaia] KevinGrandon:bug_1156089_fl_jshint > mozilla-b2g:master
Looks good. Thanks for doing this, Kevin, and sorry the review took so long.
Attachment #8594524 -
Flags: review?(dflanagan) → review+
Assignee | ||
Comment 4•10 years ago
|
||
No worries, thank you for the review! I've responded to your comment on github, happy to restructure following the landing if you would like to do so.
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 5•10 years ago
|
||
Autolander could not locate a review from a user within the suggested reviewer list. Either the patch author or the reviewer should be in the suggested reviewer list.
Assignee | ||
Comment 6•10 years ago
|
||
Oops, we have no forward lock component, but since you are a peer of gallery, and this was essentially implemented for media - I'm going to cheat and move this to the gallery component.
Component: Gaia::FMRadio → Gaia::Gallery
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 7•10 years ago
|
||
Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/42193b8356d6eaedd00493c25c3846ec98fe204a
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•