Merge pull request #693 from Praqma/fix-tmp-cleanup

Fix broken .helmsman-tmp cleanup caused by os.Exit not respectng cleanup defer
Praqma · Jul 27, 2022 · f337837 · f337837
2 parents a849965 + 355c6e0
commit f337837
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/cmd/helmsman/main.go b/cmd/helmsman/main.go
@@ -2,8 +2,10 @@ package main
 
 import (
 	"github.com/Praqma/helmsman/internal/app"
+	"os"
 )
 
 func main() {
-	app.Main()
+	exitCode := app.Main()
+	os.Exit(exitCode)
 }
diff --git a/internal/app/main.go b/internal/app/main.go
@@ -32,7 +32,7 @@ func init() {
 }
 
 // Main is the app main function
-func Main() {
+func Main() int {
 	var s State
 
 	flags.parse()
@@ -141,5 +141,5 @@ func Main() {
 		exitCode = exitCodeSucceedWithChanges
 	}
 
-	os.Exit(exitCode)
+	return exitCode
 }