oss/forgejo-runner
oss/forgejo-runner
1
0
Fork 0
mirror of https://code.forgejo.org/forgejo/runner.git synced 2025-09-15 18:57:01 +00:00
Code Wiki Activity

fix: set composite outputs on failure (#945)

Browse source 
fix: conclusion and outcome after error with failure condition
fix: continue-on-error doesn't work correctly for composite actions
This commit is contained in:
ChristopherHX 2022-01-21 17:08:30 +01:00 • committed by GitHub
parent 61c66784dd
commit 2ee208a335
5 changed files with 32 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

5
act/runner/step_context.go
View file

@ -722,9 +722,6 @@ func (sc *StepContext) execAsComposite(ctx context.Context, step *model.Step, _
compositerc.Inputs = inputs
compositerc.ExprEval = compositerc.NewExpressionEvaluator()
err = compositerc.CompositeExecutor()(ctx)
if err != nil {
return err
}
// Map outputs to parent rc
eval = (&StepContext{
@ -744,7 +741,7 @@ func (sc *StepContext) execAsComposite(ctx context.Context, step *model.Step, _
backup.Env[k] = v
}
}
return nil
return err
}
func (sc *StepContext) populateEnvsFromInput(action *model.Action, rc *RunContext) {