Skip to content

Commit 7d34d02

Browse files
committed
Revert "Only execute post for actions that have one"
This reverts commit 9675941.
1 parent 87054ff commit 7d34d02

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

src/Runner.Worker/ActionManager.cs

+9-12
Original file line numberDiff line numberDiff line change
@@ -269,19 +269,16 @@ public sealed class ActionManager : RunnerService, IActionManager
269269
}
270270
else if (depth > 0)
271271
{
272-
var definition = LoadAction(executionContext, action);
273-
if (definition.Data.Execution.HasPost) {
274-
// if we're in a composite action and haven't loaded the local action yet
275-
// we assume it has a post step
276-
if (!_cachedEmbeddedPostSteps.ContainsKey(parentStepId))
277-
{
278-
// If we haven't done so already, add the parent to the post steps
279-
_cachedEmbeddedPostSteps[parentStepId] = new Stack<Pipelines.ActionStep>();
280-
}
281-
// Clone action so we can modify the condition without affecting the original
282-
var clonedAction = action.Clone() as Pipelines.ActionStep;
283-
_cachedEmbeddedPostSteps[parentStepId].Push(clonedAction);
272+
// if we're in a composite action and haven't loaded the local action yet
273+
// we assume it has a post step
274+
if (!_cachedEmbeddedPostSteps.ContainsKey(parentStepId))
275+
{
276+
// If we haven't done so already, add the parent to the post steps
277+
_cachedEmbeddedPostSteps[parentStepId] = new Stack<Pipelines.ActionStep>();
284278
}
279+
// Clone action so we can modify the condition without affecting the original
280+
var clonedAction = action.Clone() as Pipelines.ActionStep;
281+
_cachedEmbeddedPostSteps[parentStepId].Push(clonedAction);
285282
}
286283
}
287284
}

0 commit comments

Comments
 (0)