From 8032969c69459553abdeee87e8469d02e1f7c62c Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Sun, 27 Jan 2019 08:58:16 -0800
Subject: [PATCH] tools: make trailing commas consistent in .eslintrc
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

.eslintrc has lots of trailing commas and lots of places where there
aren't trailing commas. Use trailing commas throughout the files for
consistency.

PR-URL: https://github.com/nodejs/node/pull/25739
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Gus Caplan <me@gus.host>
---
 .eslintrc.js | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/.eslintrc.js b/.eslintrc.js
index 5f336754776815..e027e5b7fd479b 100644
--- a/.eslintrc.js
+++ b/.eslintrc.js
@@ -64,11 +64,11 @@ module.exports = {
         // start with something that looks like a variable name or code.
         ignorePattern: '^.{0,50}$|^ [a-z]+ ?[0-9A-Z_.(/=:[#-]',
         ignoreInlineComments: true,
-        ignoreConsecutiveComments: true
+        ignoreConsecutiveComments: true,
       },
       block: {
-        ignorePattern: '.*'
-      }
+        ignorePattern: '.*',
+      },
     }],
     'comma-dangle': ['error', 'only-multiline'],
     'comma-spacing': 'error',
@@ -168,7 +168,7 @@ module.exports = {
       {
         property: '__defineSetter__',
         message: '__defineSetter__ is deprecated.',
-      }
+      },
     ],
     // If this list is modified, please copy the change to lib/.eslintrc.yaml
     // and test/.eslintrc.yaml.
@@ -176,11 +176,11 @@ module.exports = {
       'error',
       {
         selector: "CallExpression[callee.object.name='assert'][callee.property.name='deepStrictEqual'][arguments.2.type='Literal']",
-        message: 'Do not use a literal for the third argument of assert.deepStrictEqual()'
+        message: 'Do not use a literal for the third argument of assert.deepStrictEqual()',
       },
       {
         selector: "CallExpression[callee.object.name='assert'][callee.property.name='doesNotThrow']",
-        message: 'Please replace `assert.doesNotThrow()` and add a comment next to the code instead.'
+        message: 'Please replace `assert.doesNotThrow()` and add a comment next to the code instead.',
       },
       {
         selector: "CallExpression[callee.object.name='assert'][callee.property.name='rejects'][arguments.length<2]",
@@ -188,7 +188,7 @@ module.exports = {
       },
       {
         selector: "CallExpression[callee.object.name='assert'][callee.property.name='strictEqual'][arguments.2.type='Literal']",
-        message: 'Do not use a literal for the third argument of assert.strictEqual()'
+        message: 'Do not use a literal for the third argument of assert.strictEqual()',
       },
       {
         selector: "CallExpression[callee.object.name='assert'][callee.property.name='throws'][arguments.1.type='Literal']:not([arguments.1.regex])",
@@ -209,7 +209,7 @@ module.exports = {
       {
         selector: 'ThrowStatement > CallExpression[callee.name=/Error$/]',
         message: 'Use `new` keyword when throwing an `Error`.',
-      }
+      },
     ],
     /* eslint-enable max-len */
     'no-return-await': 'error',
@@ -263,7 +263,7 @@ module.exports = {
     'space-unary-ops': 'error',
     'spaced-comment': ['error', 'always', {
       'block': { 'balanced': true },
-      'exceptions': ['-']
+      'exceptions': ['-'],
     }],
     'strict': ['error', 'global'],
     'symbol-description': 'error',