What is the purpose of setting a `message` for `setResult(TaskResult.Succeeded, "Success");` calls?
Ethan-Arrowood opened this issue · 4 comments
Ethan-Arrowood commented
Trying to better understand the setResult
api. Based on the docs, I'm wondering what is the purpose of the second argument, message
, when TaskResult.Succeeded
is used? Should it be optional for that enum?
ivanduplenskikh commented
Hi @Ethan-Arrowood, it's a good remark.
I made a new PR with proper overloads for function.
Ethan-Arrowood commented
Could you take a look at #929 too? I opened this one for a similar thing
ivanduplenskikh commented
@Ethan-Arrowood, I'm closing this issue for now so #941 has been merged.
I left a review comment in #929.
Ethan-Arrowood commented
I don't see the review comment on #929