Merge pull request #15 from TobyMosque/master

EEParker · web-flow · commit 34eba7dcbfdb · 2019-04-09T11:05:14.000-05:00
Yarn on Windows
diff --git a/src/VueCliMiddleware/Util/ScriptRunner.cs b/src/VueCliMiddleware/Util/ScriptRunner.cs
@@ -49,8 +49,8 @@ public ScriptRunner(string workingDirectory, string scriptName, string arguments
                 // On Windows, the NPM executable is a .cmd file, so it can't be executed
                 // directly (except with UseShellExecute=true, but that's no good, because
                 // it prevents capturing stdio). So we need to invoke it via "cmd /c".
+                completeArguments = $"/c {npmExe} {completeArguments}";
                 npmExe = "cmd";
-                completeArguments = $"/c npm {completeArguments}";
             }
 
             var processStartInfo = new ProcessStartInfo(npmExe)

Original file line number	Diff line number	Diff line change
`@@ -49,8 +49,8 @@ public ScriptRunner(string workingDirectory, string scriptName, string arguments`
`49`	`49`	`// On Windows, the NPM executable is a .cmd file, so it can't be executed`
`50`	`50`	`// directly (except with UseShellExecute=true, but that's no good, because`
`51`	`51`	`// it prevents capturing stdio). So we need to invoke it via "cmd /c".`
	`52`	`+ completeArguments = $"/c {npmExe} {completeArguments}";`
`52`	`53`	`npmExe = "cmd";`
`53`		`- completeArguments = $"/c npm {completeArguments}";`
`54`	`54`	`}`
`55`	`55`
`56`	`56`	`var processStartInfo = new ProcessStartInfo(npmExe)`