Skip to content

Commit 71ee501

Browse files
committed
fix(execution): don't exclude invoked targets from skipping (#1414)
1 parent e6df653 commit 71ee501

File tree

2 files changed

+2
-11
lines changed

2 files changed

+2
-11
lines changed

source/Nuke.Build.Tests/BuildExecutorTest.cs

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -36,21 +36,12 @@ public void TestDefault()
3636
}
3737

3838
[Fact]
39-
public void TestParameterSkipped_AllWithoutInvoked()
39+
public void TestParameterSkipped_All()
4040
{
4141
ExecuteBuild(skippedTargets: new ExecutableTarget[0]);
4242
AssertSkipped(A, B, C);
4343
}
4444

45-
[Fact]
46-
public void TestParameterSkipped_AllWithInvoked()
47-
{
48-
C.Invoked = true;
49-
ExecuteBuild(skippedTargets: new ExecutableTarget[0]);
50-
AssertSucceeded(C);
51-
AssertSkipped(A, B);
52-
}
53-
5445
[Fact]
5546
public void TestParameterSkipped_Single()
5647
{

source/Nuke.Build/Execution/BuildExecutor.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -181,7 +181,7 @@ string Format(string condition)
181181

182182
private static void MarkTargetSkipped(INukeBuild build, ExecutableTarget target, string reason = null)
183183
{
184-
if (target.Invoked || target.Status != ExecutionStatus.Scheduled)
184+
if (target.Status != ExecutionStatus.Scheduled)
185185
return;
186186

187187
target.Status = ExecutionStatus.Skipped;

0 commit comments

Comments
 (0)