Closed Bug 1156610 Opened 7 years ago Closed 7 years ago
Fix remaining communications jshint issues
Which means removing these files from xfail.list: https://github.com/mozilla-b2g/gaia/blob/0645bbed4d6cbd8064652eebafe011edc3e417fd/build/jshint/xfail.list#L1-L7 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!
Hey Francisco - I'm looking at trying to have all of these jshint failures cleaned up in the next few weeks. Just wondering if you wanted to take this before I jumped on it. If not I'll gladly take this and maybe request a review from you. Thanks!
Or Jose - would this be something that you would want to work on?
Comment on attachment 8597701 [details] [review] [gaia] arcturus:bug-1156610 > mozilla-b2g:master Hei Kevin, can you take a quick look?
Attachment #8597701 - Flags: review?(kgrandon)
Comment on attachment 8597701 [details] [review] [gaia] arcturus:bug-1156610 > mozilla-b2g:master Awesome, thank you! I also left a small comment on github which you may want to address before landing.
Attachment #8597701 - Flags: review?(kgrandon) → review+
Thanks Kevin, left a comment on GH about disabling the check unused variables. And landed this here: https://github.com/mozilla-b2g/gaia/commit/27de2db07cf29922558f5fdf7b68bc44aba778e8
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.