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. |
1 reply
|
@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. |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
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
task3beforetask2. With implicitthen, it would result in the unwanted execution oftask3beforetask2.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 ?
All reactions