Enable the no-debugger rule for eslint

RESOLVED FIXED in Firefox 53

Status

()

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: jaws, Assigned: jaws)

Tracking

unspecified
mozilla53
Points:
---

Firefox Tracking Flags

(firefox53 fixed)

Details

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 2

2 years 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

2 years 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

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c8c8c4f545f7
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years 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.