Enable additional rules that node either already adheres to or it makes sense to do so going forward: for-direction, accessor-pairs, no-lonely-if and symbol-description. Fix all instances of no-lonely-if in lib & test and disable accessor-pairs in test-util-inspect. PR-URL: https://github.com/nodejs/node/pull/16243 Refs: https://eslint.org/docs/rules/for-direction Refs: https://eslint.org/docs/rules/accessor-pairs Refs: https://eslint.org/docs/rules/no-lonely-if Refs: https://eslint.org/docs/rules/symbol-description Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Refael Ackermann <refack@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
17 lines
427 B
YAML
17 lines
427 B
YAML
## Test-specific linter rules
|
|
|
|
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
|
|
prefer-assert-iferror: error
|
|
prefer-assert-methods: error
|
|
prefer-common-mustnotcall: error
|
|
crypto-check: error
|
|
inspector-check: error
|
|
## common module is mandatory in tests
|
|
required-modules: [error, common]
|