From d68fb89f32f6f74585bfb160cede4e4f7f55cee4 Mon Sep 17 00:00:00 2001 From: yxxhero Date: Sun, 15 Mar 2026 10:53:01 +0800 Subject: [PATCH] fix: add IncludeNeeds to all ChartPrepareOptions - Add IncludeNeeds to diff, template, lint, unittest, sync, apply ChartPrepareOptions - All ConfigProviders already have DAGConfig embedded which includes IncludeNeeds() Signed-off-by: yxxhero --- pkg/app/app.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/pkg/app/app.go b/pkg/app/app.go index 2966b8dc..3c60fbd4 100644 --- a/pkg/app/app.go +++ b/pkg/app/app.go @@ -169,6 +169,7 @@ func (a *App) Diff(c DiffConfigProvider) error { IncludeCRDs: &includeCRDs, Validate: c.Validate(), Concurrency: c.Concurrency(), + IncludeNeeds: c.IncludeNeeds(), IncludeTransitiveNeeds: c.IncludeTransitiveNeeds(), }, func() { msg, matched, affected, errs = a.diff(run, c) @@ -240,6 +241,7 @@ func (a *App) Template(c TemplateConfigProvider) error { SkipCleanup: c.SkipCleanup(), Validate: c.Validate(), Concurrency: c.Concurrency(), + IncludeNeeds: c.IncludeNeeds(), IncludeTransitiveNeeds: c.IncludeTransitiveNeeds(), Set: c.Set(), Values: c.Values(), @@ -319,6 +321,7 @@ func (a *App) Lint(c LintConfigProvider) error { SkipDeps: c.SkipDeps(), SkipCleanup: c.SkipCleanup(), Concurrency: c.Concurrency(), + IncludeNeeds: c.IncludeNeeds(), IncludeTransitiveNeeds: c.IncludeTransitiveNeeds(), }, func() { ok, lintErrs, errs = a.lint(run, c) @@ -360,6 +363,7 @@ func (a *App) Unittest(c UnittestConfigProvider) error { SkipDeps: c.SkipDeps(), SkipCleanup: c.SkipCleanup(), Concurrency: c.Concurrency(), + IncludeNeeds: c.IncludeNeeds(), IncludeTransitiveNeeds: c.IncludeTransitiveNeeds(), }, func() { ok, unittestErrs, errs = a.unittest(run, c) @@ -419,6 +423,7 @@ func (a *App) Sync(c SyncConfigProvider) error { WaitRetries: c.WaitRetries(), WaitForJobs: c.WaitForJobs(), IncludeCRDs: &includeCRDs, + IncludeNeeds: c.IncludeNeeds(), IncludeTransitiveNeeds: c.IncludeTransitiveNeeds(), Validate: c.Validate(), Concurrency: c.Concurrency(), @@ -457,6 +462,7 @@ func (a *App) Apply(c ApplyConfigProvider) error { SkipCleanup: c.SkipCleanup(), Validate: c.Validate(), Concurrency: c.Concurrency(), + IncludeNeeds: c.IncludeNeeds(), IncludeTransitiveNeeds: c.IncludeTransitiveNeeds(), }, func() { matched, updated, es := a.apply(run, c)