Merge pull request #46 from suzuki-shunsuke/fix/fix-45

fix: set task before evaluate task.output
suzuki-shunsuke · Oct 12, 2020 · f3fbefb · f3fbefb
2 parents e458f76 + c364a77
commit f3fbefb
Showing 1 changed file with 1 addition and 0 deletions.
diff --git a/pkg/controller/phase.go b/pkg/controller/phase.go
@@ -236,6 +236,7 @@ func (phase *Phase) runTask(ctx context.Context, idx int, task Task, params Para
 		return
 	}
 	task.Result.Status = constant.Succeeded
+	paramsPhase.Tasks.Set(idx, task)
 	output, err := task.Config.Output.Run(params.ToExpr())
 	if err != nil {
 		task.Result.Status = constant.Failed