[Stingray] jshint failure in tv browser

RESOLVED FIXED

Status

Firefox OS
General
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: chens, Assigned: chens)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [ft:conndevices][ETA:1/23])

Attachments

(1 attachment)

(Assignee)

Description

3 years ago
We have to fix jshint failures to prevent code breaking in tv's broswer
(Assignee)

Updated

3 years ago
Blocks: 1120340
No longer depends on: 1120340
Whiteboard: [ft:conndevices][ETA:1/23]
(Assignee)

Comment 1

3 years ago
Created attachment 8551051 [details] [review]
Pull request to master

Hi Yifan,

This patch fixes jshint failures in tv's browser app, would you review this patch and give some feedback? thanks!
Assignee: nobody → shchen
Attachment #8551051 - Flags: review?(yliao)
(Assignee)

Comment 2

3 years ago
Comment on attachment 8551051 [details] [review]
Pull request to master

Hi John, 

I think it's a trivial patch to fix jshint error on browser test, could either Yifan or you review this patch and give some feedback?
Attachment #8551051 - Flags: review?(im)
Comment on attachment 8551051 [details] [review]
Pull request to master

looks good to me.
Attachment #8551051 - Flags: review?(im) → review+
(Assignee)

Updated

3 years ago
Attachment #8551051 - Flags: review?(yliao)
(Assignee)

Comment 4

3 years ago
master: https://github.com/mozilla-b2g/gaia/commit/5b03150a15bfb7a967e1a3b1523f35c711592e2a
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.