Skip to content

Commit

Permalink
[eslint config] [base] disable the deprecated no-spaced-func rule
Browse files Browse the repository at this point in the history
This rule was deprecated since ESLint v3.3.0 and replaced by the
`func-call-spacing` rule.
https://eslint.org/docs/latest/rules/no-spaced-func

`func-call-spacing` was enabled in this package long ago:
airbnb/javascript@27dcb99

So there's no need for it to stay in the ruleset.

Besides, it is causing some tricky issues when overriding the rules for
TypeScript support (iamturns/eslint-config-airbnb-typescript#246)
VenusPR committed Aug 8, 2022

Verified

This commit was signed with the committer’s verified signature.
eljamm Fedi Jamoussi
1 parent 5fd7c0d commit 274cb5f
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion packages/eslint-config-airbnb-base/rules/style.js
Original file line number Diff line number Diff line change
@@ -351,7 +351,8 @@ module.exports = {
],

// disallow space between function identifier and application
'no-spaced-func': 'error',
// deprecated in favor of func-call-spacing
'no-spaced-func': 'off',

// disallow tab characters entirely
'no-tabs': 'error',

0 comments on commit 274cb5f

Please sign in to comment.