|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
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 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...?
(In reply to Andrew Swan [:aswan] from comment #2) > whoops, how did that ever get past review...? Well it won't in the future :)
Pushed by email@example.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