amp-github-apps icon indicating copy to clipboard operation
amp-github-apps copied to clipboard

Published 20 hours ago • verified publisher icon ampproject

Delete outdated comment when a PR update removes all notify owners

Open rcebulko opened this issue 4 years ago • 0 comments

Context: https://github.com/ampproject/amphtml/pull/30230#issuecomment-705704728 Requires:

  • [ ] Add deleteComment to owners/src/api/github.js#GitHub
  • [ ] Add logic within owners/src/notifier.js#createNotificationComment to delete the existing comment if the notification set is empty
  • [ ] Improve logic for identifying Owners Bot comment vs. other bot comments

rcebulko avatar Oct 08 '20 17:10 rcebulko