Removed npm-which because it was causing unexpected behavior. Will try to find another solution for Windows

Author: JamesMessinger

lib/index.js

@@ -201,25 +201,15 @@ function git (manifests, options) {
  *
  * @param {string}    command - The command to run
  * @param {string[]}  args - An array of arguments to pass
- * @param {object} [opts] - Options to pass to spawnSync
- * @param {boolean} [opts.skipCustomErrorLog] - Use to skip logging the error when using try-catch
  */
-function exec (command, args, opts) {
-  opts = opts || {};
-  let skipCustomErrorLog = opts.skipCustomErrorLog;
-  delete opts.skipCustomErrorLog;
-  // use which to make sure that node spawn can find correct executable, at the moment this is used as
-  // a windows fallback, until https://github.com/libuv/libuv/pull/358 is fixed
-  const which = require('npm-which')(process.cwd());
-  let pathToCommand = which.sync(command);
-  let result = spawnSync(pathToCommand, args, opts);
+function exec (command, args) {
+  let result = spawnSync(command, args);
+
   if (result.status || result.error) {
-    if (!skipCustomErrorLog) {
-      console.error(
-        chalk.bold.red('Error running command:'),
-        chalk.reset.magenta(command, args.join(' '))
-      );
-    }
+    console.error(
+      chalk.bold.red('Error running command:'),
+      chalk.reset.magenta(command, args.join(' '))
+    );
 
     let err = result.error;
     if (!result.error) {