Fix: Fix Node 4 by not using .includes() (#3654)

BYK · cpojer · commit 2cb5e6c6b23c · 2017-06-16T19:33:26.000+01:00
diff --git a/src/util/execute-lifecycle-script.js b/src/util/execute-lifecycle-script.js
@@ -141,7 +141,7 @@ export async function executeLifecycleScript(
   // Add global bin folder if it is not present already, as some packages depend
   // on a globally-installed version of node-gyp.
   const globalBin = getGlobalBinFolder(config, {});
-  if (!pathParts.includes(globalBin)) {
+  if (pathParts.indexOf(globalBin) === -1) {
     pathParts.unshift(globalBin);
   }
 

Original file line number	Diff line number	Diff line change
`@@ -141,7 +141,7 @@ export async function executeLifecycleScript(`
`141`	`141`	`// Add global bin folder if it is not present already, as some packages depend`
`142`	`142`	`// on a globally-installed version of node-gyp.`
`143`	`143`	`const globalBin = getGlobalBinFolder(config, {});`
`144`		`- if (!pathParts.includes(globalBin)) {`
	`144`	`+ if (pathParts.indexOf(globalBin) === -1) {`
`145`	`145`	`pathParts.unshift(globalBin);`
`146`	`146`	`}`
`147`	`147`