lib: enforce use of trailing commas

PR-URL: #46881
Reviewed-By: Debadree Chatterjee <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Mohammed Keyvanzadeh <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Jacob Smith <[email protected]>

Author: aduh95

lib/.eslintrc.yaml

@@ -2,13 +2,6 @@ env:
   es6: true
 
 rules:
-  comma-dangle: [error, {
-    arrays: always-multiline,
-    exports: always-multiline,
-    functions: always-multiline,
-    imports: always-multiline,
-    objects: only-multiline,
-  }]
   prefer-object-spread: error
   no-buffer-constructor: error
   no-mixed-operators:
@@ -249,30 +242,3 @@ globals:
   module: false
   internalBinding: false
   primordials: false
-overrides:
-  - files:
-      - ./*/*.js
-      - ./*.js
-      - ./internal/child_process/*.js
-      - ./internal/cluster/*.js
-      - ./internal/debugger/*.js
-      - ./internal/events/*.js
-      - ./internal/fs/*.js
-      - ./internal/modules/*.js
-      - ./internal/per_context/*.js
-      - ./internal/perf/*.js
-      - ./internal/policy/*.js
-      - ./internal/process/*.js
-      - ./internal/readline/*.js
-      - ./internal/readme.md
-      - ./internal/repl/*.js
-      - ./internal/source_map/*.js
-      - ./internal/streams/*.js
-      - ./internal/test/*.js
-      - ./internal/test_runner/**/*.js
-      - ./internal/tls/*.js
-      - ./internal/util/parse_args/*.js
-      - ./internal/watch_mode/*.js
-      - ./internal/webstreams/*.js
-    rules:
-      comma-dangle: [error, always-multiline]

lib/internal/assert/assertion_error.js

@@ -39,7 +39,7 @@ const kReadableOperator = {
     'Expected "actual" not to be reference-equal to "expected":',
   notDeepEqual: 'Expected "actual" not to be loosely deep-equal to:',
   notIdentical: 'Values have same structure but are not reference-equal:',
-  notDeepEqualUnequal: 'Expected values not to be loosely deep-equal:'
+  notDeepEqualUnequal: 'Expected values not to be loosely deep-equal:',
 };
 
 // Comparing short primitives should just show === / !== instead of using the
@@ -331,11 +331,11 @@ class AssertionError extends Error {
       stackStartFn,
       details,
       // Compatibility with older versions.
-      stackStartFunction
+      stackStartFunction,
     } = options;
     let {
       actual,
-      expected
+      expected,
     } = options;
 
     const limit = Error.stackTraceLimit;
@@ -430,7 +430,7 @@ class AssertionError extends Error {
       value: 'AssertionError [ERR_ASSERTION]',
       enumerable: false,
       writable: true,
-      configurable: true
+      configurable: true,
     });
     this.code = 'ERR_ASSERTION';
     if (details) {
@@ -479,7 +479,7 @@ class AssertionError extends Error {
     const result = inspect(this, {
       ...ctx,
       customInspect: false,
-      depth: 0
+      depth: 0,
     });
 
     // Reset the properties after inspection.