Skip to content

Update existing comment if no issues found#122

Merged
43081j merged 2 commits intoe18e:mainfrom
delucis:chris/fix-121
Feb 4, 2026
Merged

Update existing comment if no issues found#122
43081j merged 2 commits intoe18e:mainfrom
delucis:chris/fix-121

Conversation

@delucis
Copy link
Contributor

@delucis delucis commented Feb 4, 2026

Closes #121

If the action previously created a PR comment but there are no longer issues, this change makes it update the comment to the following content:

e18e dependency diff action

No dependency warnings found.

Previously, the action would do nothing leaving stale warnings in the PR.

Definitely open to input on the style of that message, if it matches the other output, if there’s specific emoji usage that could help, etc.

@ghostdevv
Copy link

ghostdevv commented Feb 4, 2026

this made me think that we could use the minimise comment feature to mark the comment as resolved 👀

https://docs.github.com/en/graphql/reference/mutations#minimizecomment

Edit: I think it only works on issues :((

Co-authored-by: James Garbutt <43081j@users.noreply.github.com>
@43081j 43081j merged commit 72f0719 into e18e:main Feb 4, 2026
3 of 4 checks passed
@delucis delucis deleted the chris/fix-121 branch February 4, 2026 20:13
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.

Clear out PR comment when issues are resolved

3 participants