Merge pull request #252 from wordpress-mobile/fix/remove-npm-login-shell

Remove login shell flag so that environment variables are used.
wordpress-mobile · Apr 29, 2024 · 344105d · 344105d
2 parents e8d5919 + 7b9025c
commit 344105d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/gbm-cli/pkg/shell/npm.go b/gbm-cli/pkg/shell/npm.go
@@ -24,9 +24,9 @@ func switchNodeCmd(cmds ...string) *exec.Cmd {
 		nvmCheck := exec.Command("bash", "-l", "-c", "nvm")
 		if err := nvmCheck.Run(); err != nil {
 			// Load nvm before running npm
-			return exec.Command("bash", "-l", "-c", ". $NVM_DIR/nvm.sh && "+nvmCmd)
+			return exec.Command("bash", "-c", ". $NVM_DIR/nvm.sh && "+nvmCmd)
 		} else {
-			return exec.Command("bash", "-l", "-c", nvmCmd)
+			return exec.Command("bash", "-c", nvmCmd)
 		}
 	}