Fix jshint error for build/jsmin.js

RESOLVED WONTFIX

Status

RESOLVED WONTFIX
4 years ago
4 years ago

People

(Reporter: yurenju, Assigned: ShellHacker, Mentored)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: [good first bug][mentor-lang=zh])

Attachments

(2 attachments)

+++ This bug was initially created as a clone of Bug #1032008 +++
Mentor: yurenju.mozilla
Whiteboard: [good first bug][mentor-lang=zh]

Comment 1

4 years ago
Some guidance on how to approach this bug is available here:

https://bugzilla.mozilla.org/show_bug.cgi?id=1032045#c2

Comment 2

4 years ago
jshint (v2.5.2) didn't detect any error/warning in this file on my machine. Should this bug be closed?
Flags: needinfo?(yurenju.mozilla)
:xfq,

I removed build/jsmin.js from .jshintignore and run jshint then got 124 errors, so that is still a valid bug.
Flags: needinfo?(yurenju.mozilla)
(Assignee)

Comment 4

4 years ago
Created attachment 8465526 [details] [diff] [review]
Patch_1032020.patch
Assignee: nobody → sudheesh1995
Attachment #8465526 - Flags: review?(yurenju.mozilla)
Flags: needinfo?(yurenju.mozilla)
we have same issue as bug 1032004, this script came from other project and we would like to fix it on upstream project, so reolsved it as WONTFIX.
Status: NEW → RESOLVED
Last Resolved: 4 years ago
Flags: needinfo?(yurenju.mozilla)
Resolution: --- → WONTFIX
Attachment #8465526 - Flags: review?(yurenju.mozilla)
You need to log in before you can comment on or make changes to this bug.