From 01f40d9fbcab0a1b10038928db94f24a5280f4b6 Mon Sep 17 00:00:00 2001 From: Matthieu MOREL Date: Sat, 22 Aug 2026 10:57:33 +0200 Subject: [PATCH] Apply suggestions from code review Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Signed-off-by: Matthieu MOREL --- pkg/downloader/manager.go | 4 +--- pkg/kube/client.go | 2 +- 2 files changed, 2 insertions(+), 4 deletions(-) diff --git a/pkg/downloader/manager.go b/pkg/downloader/manager.go index 5a35cc488..c4e12e1d2 100644 --- a/pkg/downloader/manager.go +++ b/pkg/downloader/manager.go @@ -733,9 +733,7 @@ func (m *Manager) findChartURL(name, version, repoURL string, repos map[string]* var entry repo.ChartVersions entry, err = findEntryByName(name, cr) if err != nil { - // TODO: Where linting is skipped in this function we should - // refactor to remove naked returns while ensuring the same - // behavior + // TODO: Consider refactoring this function to reduce the number of returned values while preserving behavior. return url, username, password, insecureSkipTLSVerify, passCredentialsAll, caFile, certFile, keyFile, err } var ve *repo.ChartVersion diff --git a/pkg/kube/client.go b/pkg/kube/client.go index 2e85ad05d..03a2761f3 100644 --- a/pkg/kube/client.go +++ b/pkg/kube/client.go @@ -1237,7 +1237,7 @@ func upgradeClientSideFieldManager(info *resource.Info, dryRun bool, fieldValida return patched, err } -// Patch reource using server-side apply +// Patch resource using server-side apply func patchResourceServerSide(target *resource.Info, dryRun, forceConflicts bool, fieldValidationDirective FieldValidationDirective) error { helper := resource.NewHelper( target.Client,