Closed Bug 1395262 Opened 5 years ago Closed 5 years ago
replace e => console
.error(e) with console .error
59 bytes, text/x-review-board-request
We decided to do the reverse of bug 1234620: replace e => console.error(e) with plain console.error. Normally this wouldn't be worth doing, but a quick inspection showed some invalid code like .catch(e => console.error), which isn't useful at all. Also, it's trivial to fix the normal uses of this pattern.
Comment on attachment 8902798 [details] Bug 1395262 - use plain console.error in devtools promise catches; https://reviewboard.mozilla.org/r/174468/#review179634 Thank you! Worth a try push before landing, but this is nice
Attachment #8902798 - Flags: review?(bgrinstead) → review+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/18d3376156f0 use plain console.error in devtools promise catches; r=bgrins
You need to log in before you can comment on or make changes to this bug.