Merge pull request #4572 from eslint/issue4569

Fix: Ignore space before function in array start (fixes #4569)
eslint · Nov 30, 2015 · 58ee960 · 58ee960
2 parents 931e0a2 + 95d2e0b
commit 58ee960
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/rules/space-before-keywords.js b/lib/rules/space-before-keywords.js
@@ -188,7 +188,7 @@ module.exports = function(context) {
                 return;
             }
 
-            checkTokens(node, left, right, { allowedPrecedingChars: [ "(", "{" ] });
+            checkTokens(node, left, right, { allowedPrecedingChars: [ "(", "{", "[" ] });
         },
         "YieldExpression": function(node) {
             check(node, { allowedPrecedingChars: [ "(", "{" ] });

diff --git a/tests/lib/rules/space-before-keywords.js b/tests/lib/rules/space-before-keywords.js
@@ -133,6 +133,7 @@ ruleTester.run("space-before-keywords", rule, {
         { code: "; function foo () {}", options: never },
         { code: ";\nfunction foo () {}", options: never },
         // FunctionExpression
+        { code: "[function () {}]" },
         { code: "var foo = function bar () {}" },
         { code: "var foo =\nfunction bar () {}" },
         { code: "function foo () { return function () {} }" },