sindresorhus/eslint-plugin-unicorn
Inconsistency in rule option names ("checkFoo" vs "ignoreFoo")
クローズ
#881 opened on 2020/10/20
docsenhancementhelp wanted
Repository metrics
- Stars
- (5,022 個のスター)
- PR merge metrics
- (平均マージ 4h 30m) (30d で 26 merged PRs)
説明
Grepping the docs for "ignore" and "check" I find these option names:
- With "ignore":
-
expiring-todo-comments:
-
-
ignoreDatesOnPullRequests
-
-
import-index:
-
-
ignoreImports
-
- With "check":
-
prevent-abbreviations:
-
-
checkDefaultAndNamespaceImports
-
-
-
checkShorthandImports
-
-
-
checkProperties
-
-
-
checkDefaultAndNamespaceImports
-
-
-
checkVariables
-
-
-
checkFilenames
-
-
no-null:
-
-
checkStrictEquality
-
-
import-style:
-
-
checkImport
-
-
-
checkDynamicImport
-
-
-
checkExportFrom
-
-
-
checkRequire
-
-
no-keyword-prefix:
-
-
checkProperties
-
The "check" names seem to win for now, so I think all new options should be named like "checkFoo" instead of "ignoreFoo". Not sure should we and how to automatically enforce this, maybe with a prevent-abbreviations replacement.