Skip to content

Commit

Permalink
fix: non-evaluated nil values should be converted to empty strings to…
Browse files Browse the repository at this point in the history
… avoid empty interface errors in the templater
  • Loading branch information
pd93 committed Dec 21, 2023
1 parent c63d160 commit 2ce18af
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions internal/compiler/v3/compiler_v3.go
Original file line number Diff line number Diff line change
Expand Up @@ -71,6 +71,12 @@ func (c *CompilerV3) getVariables(t *taskfile.Task, call *taskfile.Call, evaluat
if err := tr.Err(); err != nil {
return err
}
// If the variable should not be evaluated, but is nil, set it to an empty string
// This stops empty interface errors when using the templater to replace values later
if !evaluateShVars && newVar.Value == nil {
result.Set(k, taskfile.Var{Value: ""})
return nil
}
// If the variable is not dynamic, we can set it and return
if !evaluateShVars || newVar.Value != nil || newVar.Sh == "" {
result.Set(k, taskfile.Var{Value: newVar.Value})
Expand Down

0 comments on commit 2ce18af

Please sign in to comment.