<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<title></title>
</head>
<body>
<div name="messageBodySection">
<div dir="auto">Ali,
<div dir="auto"><br /></div>
<div dir="auto">I’m for the option 2.a because it’s not so difficult to implement but it’ll be the best effort to handle a situation more gracefully if someone puts “publish” in both places (old syntax and advanced syntax). Over time we’ll deprecate the old “publish” completely though.</div>
</div>
</div>
<div name="messageSignatureSection"><br />
<div class="matchFont">Thanks<br />
<br />
Renat Akhmerov<br />
@Nokia</div>
</div>
<div name="messageReplySection">On 28 Aug 2019, 15:37 +0700, Ali Abdelal <maxget7@gmail.com>, wrote:<br />
<blockquote type="cite" class="spark_quote" style="margin: 5px 5px; padding-left: 10px; border-left: thin solid #1abc9c;">
<div dir="ltr">Hello,<br />
<br />
Currently, there are two "publish" fields, one in the task(regular "publish")-the scope is branch and not global,<br />
and another under "on-success", “on-error” or “on-complete”.<br />
<br />
In the current behavior, regular "publish" is ignored if there is "publish" under "on-success", “on-error” or “on-complete” [1].<br />
<br />
For example:-<br />
(a)<br />
version: '2.0'<br />
wf1:<br />
tasks:<br />
t1:<br />
publish:<br />
res_x1: 1<br />
on-success:<br />
publish:<br />
branch:<br />
res_x2: 2<br />
<br />
(b)<br />
version: '2.0'<br />
wf2:<br />
tasks:<br />
t1:<br />
publish:<br />
res_x1: 1<br />
<br />
"res_x1" won't be published in (a), but it will in (b).<br />
<br />
<br />
We can either:-<br />
<br />
1) Invalidate such syntax.<br />
2) Merge the two publishes together and if there are duplicate keys, there are two options:-<br />
a) What takes priority is what's in publish under "on-success" or “on-error” or “on-complete.<br />
b) Not allow having a duplicate.<br />
<br />
<br />
What is your opinion?<br />
And please tell us if you have other suggestions.<br />
<br />
[1] <a href="https://bugs.launchpad.net/mistral/+bug/1791449">https://bugs.launchpad.net/mistral/+bug/1791449</a><br /></div>
</blockquote>
</div>
</body>
</html>