fix: use return instead of exit in install.ps1 to avoid killing session (#61)

jrob5756 · Jason Robert · web-flow · commit 7e82953fcf21 · 2026-03-26T09:10:15.000-04:00
Co-authored-by: Jason Robert &lt;jasonrobert@microsoft.com&gt;
diff --git a/install.ps1 b/install.ps1
@@ -62,7 +62,7 @@ $existingConductor = Get-Command conductor -ErrorAction SilentlyContinue
 if ($existingConductor) {
     $currentVersion = $null
     try {
-        $versionOutput = conductor --version 2>$null
+        $versionOutput = (conductor --version 2>&1) | Out-String
         if ($versionOutput -match '(\d+\.\d+\.\d+[^ ]*)') {
             $currentVersion = $Matches[1]
         }
@@ -75,7 +75,7 @@ if ($existingConductor) {
             Write-Host ""
             Write-Host "  Run 'conductor --help' to get started."
             Write-Host ""
-            exit 0
+            return
         }
         Write-Info "Upgrading Conductor: v$currentVersion → $tagName"
     }

Original file line number	Diff line number	Diff line change
`@@ -62,7 +62,7 @@ $existingConductor = Get-Command conductor -ErrorAction SilentlyContinue`
`62`	`62`	`if ($existingConductor) {`
`63`	`63`	`$currentVersion = $null`
`64`	`64`	`try {`
`65`		`- $versionOutput = conductor --version 2>$null`
	`65`	`+ $versionOutput = (conductor --version 2>&1) \| Out-String`
`66`	`66`	`if ($versionOutput -match '(\d+\.\d+\.\d+[^ ]*)') {`
`67`	`67`	`$currentVersion = $Matches[1]`
`68`	`68`	`}`
`@@ -75,7 +75,7 @@ if ($existingConductor) {`
`75`	`75`	`Write-Host ""`
`76`	`76`	`Write-Host " Run 'conductor --help' to get started."`
`77`	`77`	`Write-Host ""`
`78`		`- exit 0`
	`78`	`+ return`
`79`	`79`	`}`
`80`	`80`	`Write-Info "Upgrading Conductor: v$currentVersion → $tagName"`
`81`	`81`	`}`