Skip to content

Comments

Delete previous comments of the action#55

Merged
sfc-gh-pvillard merged 3 commits intomainfrom
comments
Jan 20, 2026
Merged

Delete previous comments of the action#55
sfc-gh-pvillard merged 3 commits intomainfrom
comments

Conversation

@sfc-gh-pvillard
Copy link
Collaborator

@sfc-gh-pvillard sfc-gh-pvillard commented Jan 20, 2026

On a very busy PR, the action will add a comment at every commit on the PR. Each comment can possibly be very large. This change ensures that previous comments from the action are deleted and only the one for the latest execution is retained.

Copy link

@sfc-gh-lshcharbaty sfc-gh-lshcharbaty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the change.
Left a few minor comments

Copy link
Collaborator

@sfc-gh-dhandermann sfc-gh-dhandermann left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the improvement and refactored approach, looks good!

@sfc-gh-pvillard sfc-gh-pvillard merged commit 5a02275 into main Jan 20, 2026
3 checks passed
@sfc-gh-pvillard sfc-gh-pvillard deleted the comments branch January 20, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants