The default bug view has changed. See this FAQ.

Enable the no-debugger rule for eslint

RESOLVED FIXED in Firefox 53

Status

()

Toolkit
General
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: jaws, Assigned: jaws)

Tracking

unspecified
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

There is only one error reported with this rule enabled and it appears to be flotsam from when the patch was being authored.

c:\fx\toolkit\mozapps\extensions\test\browser\browser_bug581076.js
  88:5  error  Unexpected 'debugger' statement.  no-debugger (eslint)
Comment hidden (mozreview-request)

Comment 2

3 months ago
mozreview-review
Comment on attachment 8823942 [details]
Bug 1328796 - Enable the no-debugger rule for eslint and remove the one instance of 'debugger;'.

https://reviewboard.mozilla.org/r/102418/#review102784

whoops, how did that ever get past review...?
Attachment #8823942 - Flags: review?(aswan) → review+
(In reply to Andrew Swan [:aswan] from comment #2)
> whoops, how did that ever get past review...?

Well it won't in the future :)

Comment 4

3 months ago
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c8c8c4f545f7
Enable the no-debugger rule for eslint and remove the one instance of 'debugger;'. r=aswan

Comment 5

3 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c8c8c4f545f7
Status: ASSIGNED → RESOLVED
Last Resolved: 3 months ago
status-firefox53: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in before you can comment on or make changes to this bug.