-
-
Notifications
You must be signed in to change notification settings - Fork 8
Closed
Labels
enhancementNew feature or requestNew feature or request
Description
See cpp-linter/cpp-linter-action#158 (comment)
-
I think we can augment the
thread-commentsinput, wheretruewould be delete and post anew (the current behavior)falsedisable feature (as has always been)updatewould be to re-use existing comment or create a new if none exists (this is the new suggestion).
-
A
no-lgtmoption would be needed to prevent the new "LGTM" comment from getting posted (as was the behavior before v1.6.0). I have to look at the history, but I think we also used to leave an outdated comment in the thread when checks passed. We should delete our previous PR comment if checks passed and an old one exists. @shenxianpeng Do you object to making thisno-lgtmoption default to true?
If I have the time, this would also be a good opportunity to make sure we check the full list of PR comments instead of only the first 30 (or whatever the REST API payload limit is).
burgholzer and shenxianpeng
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request