plan --diff says which modules a push would leave out and why, so a module just assigned whose settings cannot compose is not shown as "nothing would change"
This commit is contained in:
@@ -1239,6 +1239,10 @@ func planCommand(ctx context.Context, args []string) error {
|
|||||||
if err != nil {
|
if err != nil {
|
||||||
return err
|
return err
|
||||||
}
|
}
|
||||||
|
// A module left out is not in the body, so the diff alone would say "nothing would change" for
|
||||||
|
// a module just assigned whose settings cannot compose — success-shaped silence. Said first, with
|
||||||
|
// why, as push and the plain plan say it (novox/hq ADR 0163, rule 6).
|
||||||
|
reportLeftOut(args[0], declared)
|
||||||
return writePlanDiff(ctx, open.inventory, args[0], body)
|
return writePlanDiff(ctx, open.inventory, args[0], body)
|
||||||
}
|
}
|
||||||
if *asJSON {
|
if *asJSON {
|
||||||
|
|||||||
Reference in New Issue
Block a user