From 23ad540a0618e2900093f00e75ac947ccdb1af91 Mon Sep 17 00:00:00 2001 From: Pierre Chalamet Date: Fri, 16 Jan 2026 14:27:23 +0100 Subject: [PATCH 1/2] force build was not enforced when mode was set --- src/Terrabuild/Core/GraphPipeline/Node.fs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/Terrabuild/Core/GraphPipeline/Node.fs b/src/Terrabuild/Core/GraphPipeline/Node.fs index 7bc25bfa..414b5dd8 100644 --- a/src/Terrabuild/Core/GraphPipeline/Node.fs +++ b/src/Terrabuild/Core/GraphPipeline/Node.fs @@ -139,8 +139,8 @@ let build (options: ConfigOptions.Options) (configuration: Configuration.Workspa // auto build by default unless force let build = - let defaultForce = if options.Force then BuildMode.Always else BuildMode.Auto - targetConfig.Build |> Option.defaultValue defaultForce + if options.Force then BuildMode.Always + else targetConfig.Build |> Option.defaultValue BuildMode.Auto let required = build = BuildMode.Always From fb65b4592b60950e65c2e1c3b9c89181a8ef59f4 Mon Sep 17 00:00:00 2001 From: Pierre Chalamet Date: Fri, 16 Jan 2026 14:32:57 +0100 Subject: [PATCH 2/2] update tests --- tests/indirect-target/results/terrabuild-debug.action.json | 4 ++-- tests/indirect-target/results/terrabuild-debug.batch.json | 2 +- tests/indirect-target/results/terrabuild-debug.cascade.json | 2 +- tests/indirect-target/results/terrabuild-debug.node.json | 4 ++-- 4 files changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/indirect-target/results/terrabuild-debug.action.json b/tests/indirect-target/results/terrabuild-debug.action.json index 48fdf0bd..523f198b 100644 --- a/tests/indirect-target/results/terrabuild-debug.action.json +++ b/tests/indirect-target/results/terrabuild-debug.action.json @@ -75,10 +75,10 @@ } ], "artifacts": "none", - "build": "auto", + "build": "always", "batch": "single", "action": "exec", - "required": false + "required": true }, "workspace/path#c:build": { "id": "workspace/path#c:build", diff --git a/tests/indirect-target/results/terrabuild-debug.batch.json b/tests/indirect-target/results/terrabuild-debug.batch.json index 77ede2a3..523f198b 100644 --- a/tests/indirect-target/results/terrabuild-debug.batch.json +++ b/tests/indirect-target/results/terrabuild-debug.batch.json @@ -75,7 +75,7 @@ } ], "artifacts": "none", - "build": "auto", + "build": "always", "batch": "single", "action": "exec", "required": true diff --git a/tests/indirect-target/results/terrabuild-debug.cascade.json b/tests/indirect-target/results/terrabuild-debug.cascade.json index 77ede2a3..523f198b 100644 --- a/tests/indirect-target/results/terrabuild-debug.cascade.json +++ b/tests/indirect-target/results/terrabuild-debug.cascade.json @@ -75,7 +75,7 @@ } ], "artifacts": "none", - "build": "auto", + "build": "always", "batch": "single", "action": "exec", "required": true diff --git a/tests/indirect-target/results/terrabuild-debug.node.json b/tests/indirect-target/results/terrabuild-debug.node.json index 5851954c..710a8d21 100644 --- a/tests/indirect-target/results/terrabuild-debug.node.json +++ b/tests/indirect-target/results/terrabuild-debug.node.json @@ -75,10 +75,10 @@ } ], "artifacts": "none", - "build": "auto", + "build": "always", "batch": "single", "action": "ignore", - "required": false + "required": true }, "workspace/path#c:build": { "id": "workspace/path#c:build",