cleanup: Remove 'ProcessDependenciesWithMerge'

Signed-off-by: George Jenkins <gvjenkins@gmail.com>
pull/13599/head
George Jenkins 9 months ago
parent dde5c2aaf0
commit 3b9cce9669

@ -246,7 +246,7 @@ func (i *Install) RunWithContext(ctx context.Context, chrt *chart.Chart, vals ma
return nil, err return nil, err
} }
if err := chartutil.ProcessDependenciesWithMerge(chrt, vals); err != nil { if err := chartutil.ProcessDependencies(chrt, vals); err != nil {
return nil, err return nil, err
} }

@ -243,7 +243,7 @@ func (u *Upgrade) prepareUpgrade(name string, chart *chart.Chart, vals map[strin
return nil, nil, err return nil, nil, err
} }
if err := chartutil.ProcessDependenciesWithMerge(chart, vals); err != nil { if err := chartutil.ProcessDependencies(chart, vals); err != nil {
return nil, nil, err return nil, nil, err
} }

@ -25,19 +25,7 @@ import (
) )
// ProcessDependencies checks through this chart's dependencies, processing accordingly. // ProcessDependencies checks through this chart's dependencies, processing accordingly.
//
// TODO: For Helm v4 this can be combined with or turned into ProcessDependenciesWithMerge
func ProcessDependencies(c *chart.Chart, v Values) error { func ProcessDependencies(c *chart.Chart, v Values) error {
if err := processDependencyEnabled(c, v, ""); err != nil {
return err
}
return processDependencyImportValues(c, false)
}
// ProcessDependenciesWithMerge checks through this chart's dependencies, processing accordingly.
// It is similar to ProcessDependencies but it does not remove nil values during
// the import/export handling process.
func ProcessDependenciesWithMerge(c *chart.Chart, v Values) error {
if err := processDependencyEnabled(c, v, ""); err != nil { if err := processDependencyEnabled(c, v, ""); err != nil {
return err return err
} }

@ -87,7 +87,7 @@ func TemplatesWithSkipSchemaValidation(linter *support.Linter, values map[string
// lint ignores import-values // lint ignores import-values
// See https://github.com/helm/helm/issues/9658 // See https://github.com/helm/helm/issues/9658
if err := chartutil.ProcessDependenciesWithMerge(chart, values); err != nil { if err := chartutil.ProcessDependencies(chart, values); err != nil {
return return
} }

Loading…
Cancel
Save