From c4ebb44d1fc3511e2ae4763dafaa59004e55c50a Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Thu, 14 Aug 2025 05:28:04 +0000 Subject: [PATCH] Simplify parseHelmVersion function to be more readable Co-authored-by: yxxhero <11087727+yxxhero@users.noreply.github.com> --- pkg/helmexec/exec.go | 10 +++------- 1 file changed, 3 insertions(+), 7 deletions(-) diff --git a/pkg/helmexec/exec.go b/pkg/helmexec/exec.go index af0ca041..e2df4423 100644 --- a/pkg/helmexec/exec.go +++ b/pkg/helmexec/exec.go @@ -70,13 +70,9 @@ func parseHelmVersion(versionStr string) (*semver.Version, error) { } // Check if version string starts with "v", if not add it - processedVersion := versionStr - if !strings.HasPrefix(strings.TrimSpace(versionStr), "v") { - // Check if it looks like a semantic version (starts with a digit) - trimmed := strings.TrimSpace(versionStr) - if len(trimmed) > 0 && (trimmed[0] >= '0' && trimmed[0] <= '9') { - processedVersion = "v" + trimmed - } + processedVersion := strings.TrimSpace(versionStr) + if !strings.HasPrefix(processedVersion, "v") { + processedVersion = "v" + processedVersion } v, err := chartify.FindSemVerInfo(processedVersion)