Which means removing these files from xfail.list: https://github.com/mozilla-b2g/gaia/blob/0645bbed4d6cbd8064652eebafe011edc3e417fd/build/jshint/xfail.list#L110-L128 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 Russ - I'm looking at trying to have all of these jshint failures cleaned up in the next few weeks. I saw that you were active on the video app recently and was wondering if you or someone on your team wanted to take this. If not I'll gladly take this and maybe request a review from you. Thanks!
I will take a run at it.
Assignee: nobody → rnicoletti
Status: NEW → ASSIGNED
Hey Russ - any chance you have cycles to work on this this week? If not I'm happy to take it, thanks!
Target Milestone: --- → 2.2 S12 (15may)
Created attachment 8598257 [details] [review] Link to github PR: https://github.com/mozilla-b2g/gaia/pull/29753
Attachment #8598257 - Flags: review?(kgrandon)
Comment on attachment 8598257 [details] [review] Link to github PR: https://github.com/mozilla-b2g/gaia/pull/29753 Very happy to see these files fixed, thank you! I left a few comments on github which you may want to address before landing. I'd recommend leaving the $ function alone for the time being. Please update the commit message to contain the bug number, and re-flag me if you need me to take another look.
Attachment #8598257 - Flags: review?(kgrandon) → review+
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.