1358948 - Remove redundant eslint "no-spaced-func" rules and standardize on new "func-call-spacing" name

Status: RESOLVED FIXED

(Developer Infrastructure :: Lint and Formatting, defect)

Description

[Chris Peterson [:cpeterson]]

The "no-spaced-func" name was deprecated in ESLint v3.3.0 and replaced by "func-call-spacing", which is already specified in the mozilla/recommended rules and some other .eslintrc.js files.

http://eslint.org/docs/rules/no-spaced-func

http://eslint.org/docs/rules/func-call-spacing

Comment 1

[Chris Peterson [:cpeterson]]

Attachment: Bug 1358948 - Remove redundant eslint "no-spaced-func" rules and standardize on new "func-call-spacing" name.

The "no-spaced-func" name was deprecated in ESLint v3.3.0 and replaced by "func-call-spacing", which is already specified in the mozilla/recommended rules and some other .eslintrc.js files. We need to specify func-call-spacing in accessible/tests/browser/.eslintrc.js because it doesn't inherit the mozilla/recommended rules.

Review commit: https://reviewboard.mozilla.org/r/132752/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/132752/

Comment 2

[Mark Banner (:standard8)]

Comment on attachment 8860790 [details]
Bug 1358948 - Remove redundant eslint "no-spaced-func" rules and standardize on new "func-call-spacing" name.

https://reviewboard.mozilla.org/r/132752/#review135702

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/bec4407439ec
Remove redundant eslint "no-spaced-func" rules and standardize on new "func-call-spacing" name. r=standard8

Comment 4

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/bec4407439ec