diff --git a/cmd/helm/root.go b/cmd/helm/root.go index 7da57c6aa..4b28e3747 100644 --- a/cmd/helm/root.go +++ b/cmd/helm/root.go @@ -234,7 +234,7 @@ func checkForExpiredRepos(repofile string) { } // parse repo file. - // Ignore the error because it is okay for a repo file to be unparseable at this + // Ignore the error because it is okay for a repo file to be unparsable at this // stage. Later checks will trap the error and respond accordingly. repoFile, err := repo.LoadFile(repofile) if err != nil { diff --git a/cmd/helm/upgrade.go b/cmd/helm/upgrade.go index 2ceb31a81..16dc0f956 100644 --- a/cmd/helm/upgrade.go +++ b/cmd/helm/upgrade.go @@ -91,7 +91,7 @@ func newUpgradeCmd(cfg *action.Configuration, out io.Writer) *cobra.Command { client.Namespace = settings.Namespace() // Fixes #7002 - Support reading values from STDIN for `upgrade` command - // Must load values AFTER determining if we have to call install so that values loaded from stdin are are not read twice + // Must load values AFTER determining if we have to call install so that values loaded from stdin are not read twice if client.Install { // If a release does not exist, install it. histClient := action.NewHistory(cfg) diff --git a/pkg/action/install.go b/pkg/action/install.go index fa5508234..275e7e2ba 100644 --- a/pkg/action/install.go +++ b/pkg/action/install.go @@ -328,7 +328,7 @@ func (i *Install) RunWithContext(ctx context.Context, chrt *chart.Chart, vals ma } } - // If Replace is true, we need to supercede the last release. + // If Replace is true, we need to supersede the last release. if i.Replace { if err := i.replaceRelease(rel); err != nil { return nil, err diff --git a/pkg/action/upgrade.go b/pkg/action/upgrade.go index 690397d4a..9a48a0626 100644 --- a/pkg/action/upgrade.go +++ b/pkg/action/upgrade.go @@ -101,7 +101,7 @@ type Upgrade struct { DisableOpenAPIValidation bool // Get missing dependencies DependencyUpdate bool - // Lock to control raceconditions when the process receives a SIGTERM + // Lock to control race conditions when the process receives a SIGTERM Lock sync.Mutex } diff --git a/pkg/chartutil/capabilities.go b/pkg/chartutil/capabilities.go index 5f57e11a5..4b7fba1e9 100644 --- a/pkg/chartutil/capabilities.go +++ b/pkg/chartutil/capabilities.go @@ -111,7 +111,7 @@ func (v VersionSet) Has(apiVersion string) bool { } func allKnownVersions() VersionSet { - // We should register the built in extension APIs as well so CRDs are + // We should register the built-in extension APIs as well so CRDs are // supported in the default version set. This has caused problems with `helm // template` in the past, so let's be safe apiextensionsv1beta1.AddToScheme(scheme.Scheme) diff --git a/pkg/chartutil/expand_test.go b/pkg/chartutil/expand_test.go index f31a3d290..b46ace01f 100644 --- a/pkg/chartutil/expand_test.go +++ b/pkg/chartutil/expand_test.go @@ -64,7 +64,7 @@ func TestExpand(t *testing.T) { t.Fatal(err) } // os.Stat can return different values for directories, based on the OS - // for Linux, for example, os.Stat alwaty returns the size of the directory + // for Linux, for example, os.Stat always returns the size of the directory // (value-4096) regardless of the size of the contents of the directory mode := expect.Mode() if !mode.IsDir() { @@ -112,7 +112,7 @@ func TestExpandFile(t *testing.T) { t.Fatal(err) } // os.Stat can return different values for directories, based on the OS - // for Linux, for example, os.Stat alwaty returns the size of the directory + // for Linux, for example, os.Stat always returns the size of the directory // (value-4096) regardless of the size of the contents of the directory mode := expect.Mode() if !mode.IsDir() { diff --git a/pkg/cli/output/output.go b/pkg/cli/output/output.go index a46c977ad..01649c812 100644 --- a/pkg/cli/output/output.go +++ b/pkg/cli/output/output.go @@ -73,7 +73,7 @@ func (o Format) Write(out io.Writer, w Writer) error { } // ParseFormat takes a raw string and returns the matching Format. -// If the format does not exists, ErrInvalidFormatType is returned +// If the format does not exist, ErrInvalidFormatType is returned func ParseFormat(s string) (out Format, err error) { switch s { case Table.String(): diff --git a/pkg/downloader/manager.go b/pkg/downloader/manager.go index 18b28dde1..5ac42dae7 100644 --- a/pkg/downloader/manager.go +++ b/pkg/downloader/manager.go @@ -174,7 +174,7 @@ func (m *Manager) Update() error { // has some information about them and, when possible, the index files // locally. // TODO(mattfarina): Repositories should be explicitly added by end users - // rather than automattic. In Helm v4 require users to add repositories. They + // rather than automatic. In Helm v4 require users to add repositories. They // should have to add them in order to make sure they are aware of the // repositories and opt-in to any locations, for security. repoNames, err = m.ensureMissingRepos(repoNames, req) @@ -249,7 +249,7 @@ func (m *Manager) downloadAll(deps []*chart.Dependency) error { destPath := filepath.Join(m.ChartPath, "charts") tmpPath := filepath.Join(m.ChartPath, "tmpcharts") - // Check if 'charts' directory is not actally a directory. If it does not exist, create it. + // Check if 'charts' directory is not actually a directory. If it does not exist, create it. if fi, err := os.Stat(destPath); err == nil { if !fi.IsDir() { return errors.Errorf("%q is not a directory", destPath) diff --git a/pkg/engine/lookup_func.go b/pkg/engine/lookup_func.go index b378ca9d6..ffb399465 100644 --- a/pkg/engine/lookup_func.go +++ b/pkg/engine/lookup_func.go @@ -112,7 +112,7 @@ func getAPIResourceForGVK(gvk schema.GroupVersionKind, config *rest.Config) (met return res, err } for _, resource := range resList.APIResources { - // if a resource contains a "/" it's referencing a subresource. we don't support suberesource for now. + // if a resource contains a "/" it's referencing a subresource. we don't support subresource for now. if resource.Kind == gvk.Kind && !strings.Contains(resource.Name, "/") { res = resource res.Group = gvk.Group diff --git a/pkg/helmpath/lazypath.go b/pkg/helmpath/lazypath.go index 22d7bf0a1..97057e2b8 100644 --- a/pkg/helmpath/lazypath.go +++ b/pkg/helmpath/lazypath.go @@ -39,7 +39,7 @@ type lazypath string func (l lazypath) path(helmEnvVar, xdgEnvVar string, defaultFn func() string, elem ...string) string { - // There is an order to checking for a path. + // There is an order to check for a path. // 1. See if a Helm specific environment variable has been set. // 2. Check if an XDG environment variable is set // 3. Fall back to a default diff --git a/pkg/kube/client.go b/pkg/kube/client.go index 540b39ee7..7f263abd7 100644 --- a/pkg/kube/client.go +++ b/pkg/kube/client.go @@ -115,7 +115,7 @@ func (c *Client) getKubeClient() (*kubernetes.Clientset, error) { func (c *Client) IsReachable() error { client, err := c.getKubeClient() if err == genericclioptions.ErrEmptyConfig { - // re-replace kubernetes ErrEmptyConfig error with a friendy error + // re-replace kubernetes ErrEmptyConfig error with a friendly error // moar workarounds for Kubernetes API breaking. return errors.New("Kubernetes cluster unreachable") } diff --git a/pkg/lint/rules/template.go b/pkg/lint/rules/template.go index 61425f92e..553c1d059 100644 --- a/pkg/lint/rules/template.go +++ b/pkg/lint/rules/template.go @@ -141,7 +141,7 @@ func Templates(linter *support.Linter, values map[string]interface{}, namespace break } - // If YAML linting fails, we sill progress. So we don't capture the returned state + // If YAML linting fails, we still progress. So we don't capture the returned state // on this linter run. linter.RunLinterRule(support.ErrorSev, fpath, validateYamlContent(err)) diff --git a/pkg/registry/client_test.go b/pkg/registry/client_test.go index 138dd4245..394be67c9 100644 --- a/pkg/registry/client_test.go +++ b/pkg/registry/client_test.go @@ -193,7 +193,7 @@ func (suite *RegistryClientTestSuite) Test_1_Push() { suite.Nil(err, "no error pulling a simple chart") // Validate the output - // Note: these digests/sizes etc may change if the test chart/prov files are modified, + // Note: these digests/sizes etc. may change if the test chart/prov files are modified, // or if the format of the OCI manifest changes suite.Equal(ref, result.Ref) suite.Equal(meta.Name, result.Chart.Meta.Name) @@ -266,7 +266,7 @@ func (suite *RegistryClientTestSuite) Test_2_Pull() { suite.Nil(err, "no error pulling a chart with prov") // Validate the output - // Note: these digests/sizes etc may change if the test chart/prov files are modified, + // Note: these digests/sizes etc. may change if the test chart/prov files are modified, // or if the format of the OCI manifest changes suite.Equal(ref, result.Ref) suite.Equal(meta.Name, result.Chart.Meta.Name)