lib: add trailing commas to all public core modules

PR-URL: #46848 Reviewed-By: Debadree Chatterjee <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Anatoli Papirovski <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Rafael Gonzaga <[email protected]> Reviewed-By: Jiawen Geng <[email protected]> Reviewed-By: Trivikram Kamat <[email protected]> Reviewed-By: Yongsheng Zhang <[email protected]>
nodejs · Apr 11, 2023 · 0f33bb0 · 0f33bb0
1 parent 17fbfb2
commit 0f33bb0
Show file tree

Hide file tree

Showing 30 changed files with 214 additions and 245 deletions.
diff --git a/lib/.eslintrc.yaml b/lib/.eslintrc.yaml
@@ -252,58 +252,27 @@ globals:
 overrides:
   - files:
       - ./*/*.js
-      - ./_*.js
-      - ./child_process.js
-      - ./cluster.js
-      - ./console.js
-      - ./constants.js
-      - ./events.js
-      - ./fs.js
-      - ./internal/assert.js
-      - ./internal/child_process.js
+      - ./*.js
       - ./internal/child_process/*.js
-      - ./internal/cli_table.js
       - ./internal/cluster/*.js
       - ./internal/debugger/*.js
-      - ./internal/event_target.js
       - ./internal/events/*.js
-      - ./internal/fixed_queue.js
-      - ./internal/freelist.js
       - ./internal/fs/*.js
-      - ./internal/heap_utils.js
-      - ./internal/http.js
-      - ./internal/idna.js
-      - ./internal/js_stream_socket.js
-      - ./internal/mime.js
       - ./internal/modules/*.js
       - ./internal/per_context/*.js
       - ./internal/perf/*.js
       - ./internal/policy/*.js
-      - ./internal/priority_queue.js
       - ./internal/process/*.js
       - ./internal/readline/*.js
       - ./internal/readme.md
-      - ./internal/repl.js
       - ./internal/repl/*.js
       - ./internal/source_map/*.js
       - ./internal/streams/*.js
-      - ./internal/structured_clone.js
       - ./internal/test/*.js
       - ./internal/test_runner/**/*.js
       - ./internal/tls/*.js
       - ./internal/util/parse_args/*.js
-      - ./internal/v8_prof_processor.js
-      - ./internal/vm.js
       - ./internal/watch_mode/*.js
-      - ./internal/webidl.js
       - ./internal/webstreams/*.js
-      - ./module.js
-      - ./process.js
-      - ./punycode.js
-      - ./repl.js
-      - ./sys.js
-      - ./test.js
-      - ./tls.js
-      - ./url.js
     rules:
       comma-dangle: [error, always-multiline]
diff --git a/lib/assert.js b/lib/assert.js
@@ -163,7 +163,7 @@ function fail(actual, expected, message, operator, stackStartFn) {
     expected,
     operator: operator === undefined ? 'fail' : operator,
     stackStartFn: stackStartFn || fail,
-    message
+    message,
   };
   const err = new AssertionError(errArgs);
   if (internalMessage) {
@@ -393,7 +393,7 @@ function innerOk(fn, argLen, value, message) {
       expected: true,
       message,
       operator: '==',
-      stackStartFn: fn
+      stackStartFn: fn,
     });
     err.generatedMessage = generatedMessage;
     throw err;
@@ -430,7 +430,7 @@ assert.equal = function equal(actual, expected, message) {
       expected,
       message,
       operator: '==',
-      stackStartFn: equal
+      stackStartFn: equal,
     });
   }
 };
@@ -454,7 +454,7 @@ assert.notEqual = function notEqual(actual, expected, message) {
       expected,
       message,
       operator: '!=',
-      stackStartFn: notEqual
+      stackStartFn: notEqual,
     });
   }
 };
@@ -477,7 +477,7 @@ assert.deepEqual = function deepEqual(actual, expected, message) {
       expected,
       message,
       operator: 'deepEqual',
-      stackStartFn: deepEqual
+      stackStartFn: deepEqual,
     });
   }
 };
@@ -500,7 +500,7 @@ assert.notDeepEqual = function notDeepEqual(actual, expected, message) {
       expected,
       message,
       operator: 'notDeepEqual',
-      stackStartFn: notDeepEqual
+      stackStartFn: notDeepEqual,
     });
   }
 };
@@ -525,7 +525,7 @@ assert.deepStrictEqual = function deepStrictEqual(actual, expected, message) {
       expected,
       message,
       operator: 'deepStrictEqual',
-      stackStartFn: deepStrictEqual
+      stackStartFn: deepStrictEqual,
     });
   }
 };
@@ -550,7 +550,7 @@ function notDeepStrictEqual(actual, expected, message) {
       expected,
       message,
       operator: 'notDeepStrictEqual',
-      stackStartFn: notDeepStrictEqual
+      stackStartFn: notDeepStrictEqual,
     });
   }
 }
@@ -572,7 +572,7 @@ assert.strictEqual = function strictEqual(actual, expected, message) {
       expected,
       message,
       operator: 'strictEqual',
-      stackStartFn: strictEqual
+      stackStartFn: strictEqual,
     });
   }
 };
@@ -594,7 +594,7 @@ assert.notStrictEqual = function notStrictEqual(actual, expected, message) {
       expected,
       message,
       operator: 'notStrictEqual',
-      stackStartFn: notStrictEqual
+      stackStartFn: notStrictEqual,
     });
   }
 };
@@ -627,7 +627,7 @@ function compareExceptionKey(actual, expected, key, message, keys, fn) {
         actual: a,
         expected: b,
         operator: 'deepStrictEqual',
-        stackStartFn: fn
+        stackStartFn: fn,
       });
       err.actual = actual;
       err.expected = expected;
@@ -639,7 +639,7 @@ function compareExceptionKey(actual, expected, key, message, keys, fn) {
       expected,
       message,
       operator: fn.name,
-      stackStartFn: fn
+      stackStartFn: fn,
     });
   }
 }
@@ -668,7 +668,7 @@ function expectedException(actual, expected, message, fn) {
         expected,
         message,
         operator: 'deepStrictEqual',
-        stackStartFn: fn
+        stackStartFn: fn,
       });
       err.operator = fn.name;
       throw err;
@@ -743,7 +743,7 @@ function expectedException(actual, expected, message, fn) {
       expected,
       message,
       operator: fn.name,
-      stackStartFn: fn
+      stackStartFn: fn,
     });
     err.generatedMessage = generatedMessage;
     throw err;
@@ -837,7 +837,7 @@ function expectsError(stackStartFn, actual, error, message) {
       expected: error,
       operator: stackStartFn.name,
       message: `Missing expected ${fnType}${details}`,
-      stackStartFn
+      stackStartFn,
     });
   }
 
@@ -886,7 +886,7 @@ function expectsNoError(stackStartFn, actual, error, message) {
       operator: stackStartFn.name,
       message: `Got unwanted ${fnType}${details}\n` +
                `Actual message: "${actual && actual.message}"`,
-      stackStartFn
+      stackStartFn,
     });
   }
   throw actual;
@@ -955,7 +955,7 @@ assert.ifError = function ifError(err) {
       expected: null,
       operator: 'ifError',
       message,
-      stackStartFn: ifError
+      stackStartFn: ifError,
     });
 
     // Make sure we actually have a stack trace!
@@ -1020,7 +1020,7 @@ function internalMatch(string, regexp, message, fn) {
       expected: regexp,
       message,
       operator: fn.name,
-      stackStartFn: fn
+      stackStartFn: fn,
     });
     err.generatedMessage = generatedMessage;
     throw err;
@@ -1064,7 +1064,7 @@ assert.strict = ObjectAssign(strict, assert, {
   equal: assert.strictEqual,
   deepEqual: assert.deepStrictEqual,
   notEqual: assert.notStrictEqual,
-  notDeepEqual: assert.notDeepStrictEqual
+  notDeepEqual: assert.notDeepStrictEqual,
 });
 
 assert.strict.strict = assert.strict;
diff --git a/lib/async_hooks.js b/lib/async_hooks.js
@@ -18,7 +18,7 @@ const {
 const {
   ERR_ASYNC_CALLBACK,
   ERR_ASYNC_TYPE,
-  ERR_INVALID_ASYNC_ID
+  ERR_INVALID_ASYNC_ID,
 } = require('internal/errors').codes;
 const { kEmptyObject } = require('internal/util');
 const {
@@ -58,7 +58,7 @@ const {
 const {
   async_id_symbol, trigger_async_id_symbol,
   init_symbol, before_symbol, after_symbol, destroy_symbol,
-  promise_resolve_symbol
+  promise_resolve_symbol,
 } = internal_async_hooks.symbols;
 
 // Get constants
@@ -251,7 +251,7 @@ class AsyncResource {
         enumerable: true,
         value: this,
         writable: true,
-      }
+      },
     });
     return bound;
   }
@@ -270,7 +270,7 @@ const storageHook = createHook({
     for (let i = 0; i < storageList.length; ++i) {
       storageList[i]._propagate(resource, currentResource, type);
     }
-  }
+  },
 });
 
 class AsyncLocalStorage {