From 34e9c7ad57307baea71cadc5a7525fd9de6e18f5 Mon Sep 17 00:00:00 2001 From: Felipe Santos Date: Sun, 23 Feb 2025 22:47:46 -0300 Subject: [PATCH] Fix some simple comments Signed-off-by: Felipe Santos --- cmd/helm/dependency.go | 4 ++-- internal/resolver/resolver.go | 1 - 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/cmd/helm/dependency.go b/cmd/helm/dependency.go index 99b9b5a05..df2bd9427 100644 --- a/cmd/helm/dependency.go +++ b/cmd/helm/dependency.go @@ -70,8 +70,8 @@ the dependency charts stored locally. The path should start with a prefix of repository: "file://../dependency_chart/nginx" If the dependency chart is retrieved locally, it is not required to have the -repository added to helm by "helm add repo". Version matching is also supported -for this case, but optional. +repository added to helm by "helm add repo". Version matching is optionally +supported for this case. ` const dependencyListDesc = ` diff --git a/internal/resolver/resolver.go b/internal/resolver/resolver.go index c95e0703a..e695a179c 100644 --- a/internal/resolver/resolver.go +++ b/internal/resolver/resolver.go @@ -94,7 +94,6 @@ func (r *Resolver) Resolve(reqs []*chart.Dependency, repoNames map[string]string v, err := semver.NewVersion(ch.Metadata.Version) if err != nil { - // If version is not set and the version from the local chart is invalid if d.Version == "" { missing = append(missing, d.Name) }