Commit fd35878
committed
Fix error in
Task's `default` template command uses the first value if the second is not defined. In this case, the second value was a
variable defined by the workflow, so it was always defined! This made the default command and the first value useless. My
interpretation is that the intent was, as is done successfully elsewhere, to allow the task user to make customizations
by defining an environment variable from the command line which will override the default value. In order to accomplish that in a clean manner, I made the following changes:
- Remove the frivolous taskfile variable, instead adding the flags directly to the template
- Hard code the `-v` flag into the command
- Add an override variable with a more relevant namego test command template1 parent 75ab8aa commit fd35878
1 file changed
+2
-2
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
28 | 28 | | |
29 | 29 | | |
30 | 30 | | |
31 | | - | |
32 | 31 | | |
33 | 32 | | |
34 | 33 | | |
| |||
71 | 70 | | |
72 | 71 | | |
73 | 72 | | |
| 73 | + | |
74 | 74 | | |
75 | | - | |
76 | 75 | | |
| 76 | + | |
77 | 77 | | |
78 | 78 | | |
79 | 79 | | |
| |||
0 commit comments