tools: require more trailing commas

All of the import and export statements in the codebase already use trailing commas, this commit adds a linter rule to enforce that. PR-URL: #46346 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Filip Skokan <panva.ip@gmail.com> Reviewed-By: Darshan Sen <raisinten@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
nodejs · Mar 3, 2023 · e4756a8 · e4756a8
1 parent 88614c5
commit e4756a8
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/benchmark/.eslintrc.yaml b/benchmark/.eslintrc.yaml
@@ -7,9 +7,9 @@ env:
 rules:
   comma-dangle: [error, {
     arrays: always-multiline,
-    exports: only-multiline,
+    exports: always-multiline,
     functions: only-multiline,
-    imports: only-multiline,
+    imports: always-multiline,
     objects: only-multiline,
   }]
   prefer-arrow-callback: error
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -4,9 +4,9 @@ env:
 rules:
   comma-dangle: [error, {
     arrays: always-multiline,
-    exports: only-multiline,
+    exports: always-multiline,
     functions: only-multiline,
-    imports: only-multiline,
+    imports: always-multiline,
     objects: only-multiline,
   }]
   prefer-object-spread: error

diff --git a/test/.eslintrc.yaml b/test/.eslintrc.yaml
@@ -88,8 +88,8 @@ overrides:
     rules:
       comma-dangle: [error, {
         arrays: always-multiline,
-        exports: only-multiline,
+        exports: always-multiline,
         functions: only-multiline,
-        imports: only-multiline,
+        imports: always-multiline,
         objects: only-multiline,
       }]