add short circuit return

scottrigby · gjenkins8 · scottrigby · commit 3cd6afe845c9 · 2025-02-21T19:00:40.000-05:00
Co-authored-by: George Jenkins &lt;gvjenkins@gmail.com&gt;
Signed-off-by: Scott Rigby &lt;scott@r6by.com&gt;
diff --git a/pkg/action/hooks.go b/pkg/action/hooks.go
@@ -181,7 +181,9 @@ func hookHasDeletePolicy(h *release.Hook, policy release.HookDeletePolicy) bool
 
 // outputLogsByPolicy outputs a pods logs if the hook policy instructs it to
 func (cfg *Configuration) outputLogsByPolicy(h *release.Hook, releaseNamespace string, policy release.HookOutputLogPolicy) error {
-	if hookHasOutputLogPolicy(h, policy) {
+	if !hookHasOutputLogPolicy(h, policy) {
+	    return nil
+	}
 		namespace, err := cfg.deriveNamespace(h, releaseNamespace)
 		if err != nil {
 			return err
