Fix: ignore comma before closing paren (fixes #11295)

eslint · Feb 11, 2019 · cbae656 · cbae656
1 parent 5eb3121
commit cbae656
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/lib/rules/comma-spacing.js b/lib/rules/comma-spacing.js
@@ -118,6 +118,10 @@ module.exports = {
                 report(reportItem, "before", tokens.left);
             }
 
+            if (tokens.right && astUtils.isClosingParenToken(tokens.right)) {
+                return;
+            }
+
             if (tokens.right && !options.after && tokens.right.type === "Line") {
                 return;
             }

diff --git a/tests/lib/rules/comma-spacing.js b/tests/lib/rules/comma-spacing.js
@@ -61,6 +61,7 @@ ruleTester.run("comma-spacing", rule, {
         "['  ,  ']",
         { code: "[`  ,  `]", parserOptions: { ecmaVersion: 6 } },
         { code: "`${[1, 2]}`", parserOptions: { ecmaVersion: 6 } },
+        { code: "fn(a, b,)", parserOptions: { ecmaVersion: 2018 } }, // #11295
         "foo(/,/, 'a')",
         "var x = ',,,,,';",
         "var code = 'var foo = 1, bar = 3;'",