Commit Graph

2 Commits

Author SHA1 Message Date
yxxhero aeefe97c29 fix: address copilot review comments in integration test
- Quote all variables to prevent word-splitting/globbing issues
- Use single-quoted trap for cleanup robustness
- Remove unused output_dir variable
- Add positive assertions for release2 and release3 in lint and diff tests

(cherry picked from commit 4128650c0a)
Signed-off-by: yxxhero <aiopsclub@163.com>
2026-03-13 13:21:48 +08:00
yxxhero 47c7b75ba3 fix: --include-needs should only include direct dependencies (#1003)
--include-needs was incorrectly including transitive dependencies when
it should only include direct dependencies per the documentation.

The issue was that ChartPrepareOptions.IncludeTransitiveNeeds was being
set to c.IncludeNeeds() instead of c.IncludeTransitiveNeeds() in the
chart preparation phase for diff, template, lint, sync, and apply commands.

This fix ensures:
- --include-needs includes only direct dependencies
- --include-transitive-needs includes both direct and transitive dependencies

Fixes #1003

Signed-off-by: yxxhero <yxxhero@users.noreply.github.com>
Signed-off-by: yxxhero <aiopsclub@163.com>
(cherry picked from commit 7a2cbeebb8)
Signed-off-by: yxxhero <aiopsclub@163.com>
2026-03-13 13:21:48 +08:00