ampproject / ampproject/amp-github-apps
Delete outdated comment when a PR update removes all notify owners
- Vorherrschende Sprache
- TypeScript
- Sterne
- 35
- Forks
- 35
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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
Beitragsleitfaden
Rechercherichtung
Beginne mit owners/src/api/github.js#GitHub und owners/src/notifier.js#createNotificationComment und prüfe anschließend den verlinkten Pull-Request-Kommentar zum bestehenden Benachrichtigungsverhalten. Erledigt ist die Aufgabe, wenn die GitHub API den vorhandenen Owners Bot-Kommentar löschen kann, sobald die Menge der Benachrichtigungen leer wird, während andere Bot-Kommentare nicht fälschlicherweise als dieser erkannt werden.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- github, typescript
- Bereich
- api, tooling
- Issue-Typ
- Feature
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100