tools: enable no-empty ESLint rule

Trott · danielleadams · commit 475be6c25c01 · 2022-03-04T09:26:51.000-05:00
Refs: https://eslint.org/docs/rules/no-empty PR-URL: nodejs#41831 Backport-PR-URL: nodejs#42160 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
diff --git a/.eslintrc.js b/.eslintrc.js
@@ -295,7 +295,6 @@ module.exports = {
     'valid-typeof': ['error', { requireStringLiterals: true }],
 
     // ESLint recommended rules that we disable
-    'no-empty': 'off',
     'no-inner-declarations': 'off',
 
     // JSDoc recommended rules that we disable
diff --git a/benchmark/.eslintrc.yaml b/benchmark/.eslintrc.yaml
@@ -5,6 +5,5 @@ env:
   es6: true
 
 rules:
-  no-empty: error
   no-var: error
   prefer-arrow-callback: error
diff --git a/test/.eslintrc.yaml b/test/.eslintrc.yaml
@@ -6,7 +6,6 @@ env:
 
 rules:
   multiline-comment-style: ["error", "separate-lines"]
-  no-empty: error
   no-var: error
   prefer-const: error
   symbol-description: off
