Closed Bug 1292042 Opened 4 years ago Closed 4 years ago

Enable the keyword-spacing rule for eslint

Categories

(Toolkit :: General, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla51
Tracking Status
firefox51 --- fixed

People

(Reporter: jaws, Assigned: jaws)

Details

Attachments

(1 file)

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: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla51
You need to log in before you can comment on or make changes to this bug.