iterative
`[send-]comment` option for custom watermark
So that when cml goes to update a comment it can determine which comment to update.
For cases where multiple comments are made or when multiple workflows run against a single PR each with a cml comment, they can better distinguish which comment to update.
Potential duplicate of / related to https://github.com/iterative/cml/issues/1076?
Ok, so if two different workflows call cml comment create --update, we'd want them to update the comments these workflows originally created, right?
What if the same github workflow is run multiple times, only ever calling cml comment create --update - should it be updating the same comment?
I think we should avoid giving users too much rope (in the form of command flags) and just define several use cases we support.
if two different workflows call
cml comment create --update, we'd want them to update the comments these workflows originally created, right?
Yes, that's the goal of this issue.
What if the same github workflow is run multiple times, only ever calling
cml comment create --update- should it be updating the same comment?
That's what it currently does, but this use case is unavoidably prone to race conditions.
It could be a lot simpler and less race-prone if we limited comment update to updating comments created by this action run only. Would that be too limiting?
It could be a lot simpler and less race-prone if we limited
comment updateto updating comments created by this action run only. Would that be too limiting?
If there were multiple comment create's in a single workflow this would make sense. Where this causes issues is with pull_request type events. Where there might be several runs of a workflow for a single PR either via multiple workflows or the user pushes additional commits retriggering the event. For this case in order to prevent extra bash scripting a single create_or_update type function is needed.
How different s going to be the wartermark? Studio needs the CML waterrmarks to be able to extract the CML report.
