mirror of
https://github.com/nodejs/node.git
synced 2024-12-01 16:10:02 +01:00
ba98d9ab2c
This commit removes the custom number-isnan ESLint rule in favor of the no-restricted-syntax rule. It also applies the rule across the entire codebase, instead of just the test/ directory. PR-URL: https://github.com/nodejs/node/pull/31211 Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: David Carlier <devnexen@gmail.com> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Richard Lau <riclau@uk.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Rich Trott <rtrott@gmail.com>
36 lines
840 B
YAML
36 lines
840 B
YAML
## Test-specific linter rules
|
|
|
|
env:
|
|
node: true
|
|
es6: true
|
|
|
|
rules:
|
|
# ECMAScript 6
|
|
# http://eslint.org/docs/rules/#ecmascript-6
|
|
no-var: error
|
|
prefer-const: error
|
|
symbol-description: off
|
|
|
|
# Custom rules in tools/eslint-rules
|
|
node-core/prefer-assert-iferror: error
|
|
node-core/prefer-assert-methods: error
|
|
node-core/prefer-common-mustnotcall: error
|
|
node-core/crypto-check: error
|
|
node-core/eslint-check: error
|
|
node-core/inspector-check: error
|
|
## common module is mandatory in tests
|
|
node-core/required-modules:
|
|
- error
|
|
- common: 'common(/index\.(m)?js)?$'
|
|
node-core/require-common-first: error
|
|
node-core/no-duplicate-requires: off
|
|
|
|
# Global scoped methods and vars
|
|
globals:
|
|
WebAssembly: false
|
|
BigInt: false
|
|
BigInt64Array: false
|
|
BigUint64Array: false
|
|
SharedArrayBuffer: false
|
|
globalThis: false
|