ampproject / ampproject/amp-github-apps
Delete outdated comment when a PR update removes all notify owners
Open
Category: Owners
P3: When Possible
Type: Feature Request
- Dominant language
- TypeScript
- Stars
- 35
- Forks
- 35
- PR merge metrics
- No merged PRs in 30d
Description
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
Contributor guide
Assessment
This issue has not been assessed yet.