Merge pull request #437 from reubenmiller/fix-unintended-warning

fix: suppress warning message when displaying help
reubenmiller · Feb 16, 2025 · aad2fba · aad2fba
2 parents 29cb4f9 + c5b6a3f
commit aad2fba
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/pkg/cmd/root/root.go b/pkg/cmd/root/root.go
@@ -1,6 +1,7 @@
 package root
 
 import (
+	"errors"
 	"fmt"
 	"io/fs"
 	"os"
@@ -194,7 +195,9 @@ func NewCmdRoot(f *cmdutil.Factory, version, buildDate string) *CmdRoot {
 			if cmdErr != nil {
 				logg, logErr := f.Logger()
 				if logg != nil && logErr == nil {
-					logg.Warnf("Check existing session failed. %s", cmdErr)
+					if !errors.Is(cmderrors.ErrHelp, cmdErr) {
+						logg.Warnf("Check existing session failed. %s", cmdErr)
+					}
 				}
 			}
 			return cmdErr