sindresorhus/eslint-plugin-unicorn
Inconsistency in rule option names ("checkFoo" vs "ignoreFoo")
已关闭
#881 创建于 2020年10月20日
docsenhancementhelp wanted
仓库指标
- 星标
- (5,022 个星标)
- PR 合并指标
- (平均合并 1天 16小时) (30 天内合并 399 个 PR)
描述
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.