BuildSize icon indicating copy to clipboard operation
BuildSize copied to clipboard

Editing the existing comment makes it harder to find

Open alexeagle opened this issue 6 years ago • 2 comments

When I publish a new commit to a PR, buildsize updates the comment it made earlier.

This is nice to reduce spaminess on the thread, but on the other hand it means the comment gets lost higher up the thread, eg. https://github.com/angular/angular/pull/22040 has some later robot comments added for each commit.

Not sure if there's a way to have both low-spam and also keep the message near the bottom. Maybe it's a moot point if we solve #31 since that way the PR status will indicate the current state of the PR.

alexeagle avatar Feb 06 '18 17:02 alexeagle

For reference (in case that PR changes) here's what it looks like now rppe6slfkgb

alexeagle avatar Feb 06 '18 17:02 alexeagle

Yeah, this is tricky... One option is to delete the old comment then post a new one, but then you lose historical context. Maybe a good idea is to edit the comment if there's just a small change, but post a new comment if the change is significant?

Daniel15 avatar May 13 '18 23:05 Daniel15