[fix] jsx-no-useless-fragment: use array-includes over `.includes…

…` for back compat
jsx-eslint · Oct 3, 2019 · 6f14e16 · 6f14e16
1 parent a1dee7c
commit 6f14e16
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/lib/rules/jsx-no-useless-fragment.js b/lib/rules/jsx-no-useless-fragment.js
@@ -4,6 +4,8 @@
 
 'use strict';
 
+const arrayIncludes = require('array-includes');
+
 const pragmaUtil = require('../util/pragma');
 const jsxUtil = require('../util/jsx');
 const docsUrl = require('../util/docsUrl');
@@ -47,8 +49,8 @@ function trimLikeReact(text) {
   const leadingSpaces = /^\s*/.exec(text)[0];
   const trailingSpaces = /\s*$/.exec(text)[0];
 
-  const start = leadingSpaces.includes('\n') ? leadingSpaces.length : 0;
-  const end = trailingSpaces.includes('\n') ? text.length - trailingSpaces.length : text.length;
+  const start = arrayIncludes(leadingSpaces, '\n') ? leadingSpaces.length : 0;
+  const end = arrayIncludes(trailingSpaces, '\n') ? text.length - trailingSpaces.length : text.length;
 
   return text.slice(start, end);
 }
@@ -92,7 +94,7 @@ module.exports = {
     function isPaddingSpaces(node) {
       return isJSXText(node) &&
         isOnlyWhitespace(node.raw) &&
-        node.raw.includes('\n');
+        arrayIncludes(node.raw, '\n');
     }
 
     /**