Skip to content

Commit ac8537d

Browse files
Trottdanielleadams
authored andcommitted
tools: enable no-empty ESLint rule
Refs: https://eslint.org/docs/rules/no-empty PR-URL: nodejs#41831 Backport-PR-URL: nodejs#42160 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Antoine du Hamel <[email protected]>
1 parent 783f8ef commit ac8537d

File tree

3 files changed

+0
-3
lines changed

3 files changed

+0
-3
lines changed

.eslintrc.js

-1
Original file line numberDiff line numberDiff line change
@@ -295,7 +295,6 @@ module.exports = {
295295
'valid-typeof': ['error', { requireStringLiterals: true }],
296296

297297
// ESLint recommended rules that we disable
298-
'no-empty': 'off',
299298
'no-inner-declarations': 'off',
300299

301300
// JSDoc recommended rules that we disable

benchmark/.eslintrc.yaml

-1
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,5 @@ env:
55
es6: true
66

77
rules:
8-
no-empty: error
98
no-var: error
109
prefer-arrow-callback: error

test/.eslintrc.yaml

-1
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@ env:
66

77
rules:
88
multiline-comment-style: ["error", "separate-lines"]
9-
no-empty: error
109
no-var: error
1110
prefer-const: error
1211
symbol-description: off

0 commit comments

Comments
 (0)