Fix: comma-dangle was confused by type annotations (fixes #7370) (#…

…7372)
eslint · Oct 17, 2016 · 681c78a · 681c78a
1 parent 7525042
commit 681c78a
Show file tree

Hide file tree

Showing 6 changed files with 4,181 additions and 6 deletions.
diff --git a/lib/rules/comma-dangle.js b/lib/rules/comma-dangle.js
@@ -171,15 +171,10 @@ module.exports = {
         function getTrailingToken(node, lastItem) {
             switch (node.type) {
                 case "ObjectExpression":
-                case "ObjectPattern":
                 case "ArrayExpression":
-                case "ArrayPattern":
                 case "CallExpression":
                 case "NewExpression":
                     return sourceCode.getLastToken(node, 1);
-                case "FunctionDeclaration":
-                case "FunctionExpression":
-                    return sourceCode.getTokenBefore(node.body, 1);
                 default: {
                     const nextToken = sourceCode.getTokenAfter(lastItem);