Replies: 2 comments 1 reply
-
We already reached a consensus to go with: do:
- myTaskName:
call: ... Like Google Workflows. Your second proposal is the same than not having names. If they are optional, noone will use them, ever. Also, it hinders fluency imo. |
Beta Was this translation helpful? Give feedback.
-
@JBBianchi I prefer the second approach (name as optional, in fact I proposed that here) , but @cdavernas prefer the first one and I do not want to press the issue, so Im willing to compromise. |
Beta Was this translation helpful? Give feedback.
-
As the result of #875, the idea of having named properties to describe tasks, such as:
has been dropped because neither JSON not YAML have guaranteed properties ordering. The workflow above could be parsed with the property
task3
beforetask2
. With implicitthen
, it would result in the unwanted execution oftask3
beforetask2
.Therefore, to insure the ordering of the tasks, we'll have to rely on an array instead. For instance (in accordance with #872 and #884):
This raises again another point already mentioned by #869 which is to leave names optional and have something like this instead:
If more flow control is needed, names can be provided:
What's your take @cdavernas @fjtirado @matthias-pichler-warrify @ricardozanini ?
Beta Was this translation helpful? Give feedback.
All reactions