Enable the keyword-spacing rule for eslint

RESOLVED FIXED in Firefox 51

Status

()

defect
RESOLVED FIXED
3 years ago
3 years ago

People

(Reporter: jaws, Assigned: jaws)

Tracking

unspecified
mozilla51
Points:
---

Firefox Tracking Flags

(firefox51 fixed)

Details

Attachments

(1 attachment)

This rule enforce spaces before and after keywords, such as "if (condition)" and "catch (ex) {}".
Comment on attachment 8777677 [details]
Bug 1292042 - Enable the keyword-spacing rule for eslint. Most of this patch was autogenerated by 'eslint --fix'. Files with .xml extension needed to be changed by hand due to limitations of custom plugin.

https://reviewboard.mozilla.org/r/69190/#review66292

rubber stamp successfully deployed!
Attachment #8777677 - Flags: review?(markh) → review+
Comment on attachment 8777677 [details]
Bug 1292042 - Enable the keyword-spacing rule for eslint. Most of this patch was autogenerated by 'eslint --fix'. Files with .xml extension needed to be changed by hand due to limitations of custom plugin.

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/69190/diff/1-2/
Pushed by jwein@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/ab1d29e62ee4
Enable the keyword-spacing rule for eslint. Most of this patch was autogenerated by 'eslint --fix'. Files with .xml extension needed to be changed by hand due to limitations of custom plugin. r=markh
https://hg.mozilla.org/mozilla-central/rev/ab1d29e62ee4
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.