fixup! fixup! feat(jsx-key): check for array related method calls with jsx elements/fragments

Author: SimonSchick

lib/rules/jsx-key.js

@@ -264,17 +264,14 @@ module.exports = {
       },
 
       // eslint-disable-next-line no-multi-str
-      'CallExpression[callee.type="MemberExpression"][callee.property.name=/^push|unshift|splice|with$/],\
-       CallExpression[callee.type="OptionalMemberExpression"][callee.property.name=/^push|unshift|splice|with$/],\
-       OptionalCallExpression[callee.type="MemberExpression"][callee.property.name=/^push|unshift|splice|with$/],\
-       OptionalCallExpression[callee.type="OptionalMemberExpression"][callee.property.name=/^push|unshift|splice|with$/]'(node) {
-        if (node.arguments.length === 0) {
-          return;
-        }
-
+      'CallExpression[callee.type="MemberExpression"][callee.property.name=/^push|unshift|splice|with|concat$/],\
+       CallExpression[callee.type="OptionalMemberExpression"][callee.property.name=/^push|unshift|splice|with|concat$/],\
+       OptionalCallExpression[callee.type="MemberExpression"][callee.property.name=/^push|unshift|splice|with|concat$/],\
+       OptionalCallExpression[callee.type="OptionalMemberExpression"][callee.property.name=/^push|unshift|splice|with|concat$/]'(node) {
         node.arguments.forEach((arg) => {
           if (arg.type === 'JSXElement' && !hasProp(arg.openingElement.attributes, 'key')) {
             report(context, messages.missingIterKey, 'missingArrayKey', { node });
+            return;
           }
           if (arg.type === 'JSXFragment') {
             report(context, messages.missingIterKey, 'missingArrayKeyUsePrag', { node });