fix: add default casess to switch statements

Signed-off-by: Brenden Ehlers <brenden@theehlers.org>

.golangci.yml

@@ -73,6 +73,9 @@ linters:
               recommendations:
                 - github.com/evanphx/json-patch/v5
 
+    exhaustive:
+      default-signifies-exhaustive: true
+
 run:
   timeout: 10m
 

internal/statusreaders/pod_status_reader.go

@@ -86,8 +86,7 @@ func podConditions(u *unstructured.Unstructured) (*status.Result, error) {
 				},
 			},
 		}, nil
-	}
+	default:
-
 		message := "Pod in progress"
 		return &status.Result{
 			Status:  status.InProgressStatus,
@@ -102,3 +101,4 @@ func podConditions(u *unstructured.Unstructured) (*status.Result, error) {
 			},
 		}, nil
 	}
+}

pkg/cmd/repo_list.go

@@ -103,12 +103,12 @@ func (r *repoListWriter) encodeByFormat(out io.Writer, format output.Format) err
 		return output.EncodeJSON(out, repolist)
 	case output.YAML:
 		return output.EncodeYAML(out, repolist)
-	}
+	default:
-
 		// Because this is a non-exported function and only called internally by
 		// WriteJSON and WriteYAML, we shouldn't get invalid types
 		return nil
 	}
+}
 
 // Returns all repos from repos, except those with names matching ignoredRepoNames
 // Inspired by https://stackoverflow.com/a/28701031/893211

pkg/cmd/search_hub.go

@@ -190,9 +190,10 @@ func (h *hubSearchWriter) encodeByFormat(out io.Writer, format output.Format) er
 		return output.EncodeJSON(out, chartList)
 	case output.YAML:
 		return output.EncodeYAML(out, chartList)
-	}
+	default:
-
 		// Because this is a non-exported function and only called internally by
 		// WriteJSON and WriteYAML, we shouldn't get invalid types
 		return nil
 	}
+
+}

pkg/cmd/search_repo.go

@@ -260,12 +260,12 @@ func (r *repoSearchWriter) encodeByFormat(out io.Writer, format output.Format) e
 		return output.EncodeJSON(out, chartList)
 	case output.YAML:
 		return output.EncodeYAML(out, chartList)
-	}
+	default:
-
 		// Because this is a non-exported function and only called internally by
 		// WriteJSON and WriteYAML, we shouldn't get invalid types
 		return nil
 	}
+}
 
 // Provides the list of charts that are part of the specified repo, and that starts with 'prefix'.
 func compListChartsOfRepo(repoName string, prefix string) []string {

pkg/kube/ready.go

@@ -354,6 +354,8 @@ func (c *ReadyChecker) crdBetaReady(crd apiextv1beta1.CustomResourceDefinition)
 				// continue.
 				return true
 			}
+		default:
+			// intentionally left empty
 		}
 	}
 	return false
@@ -374,6 +376,8 @@ func (c *ReadyChecker) crdReady(crd apiextv1.CustomResourceDefinition) bool {
 				// continue.
 				return true
 			}
+		default:
+			// intentionally left empty
 		}
 	}
 	return false

pkg/kube/wait.go

@@ -333,6 +333,8 @@ func (hw *legacyWaiter) waitForPodSuccess(obj runtime.Object, name string) (bool
 		slog.Debug("pod pending", "pod", o.Name)
 	case corev1.PodRunning:
 		slog.Debug("pod running", "pod", o.Name)
+	case corev1.PodUnknown:
+		slog.Debug("pod unknown", "pod", o.Name)
 	}
 
 	return false, nil