ampproject / ampproject/amp-github-apps

Delete outdated comment when a PR update removes all notify owners

Aperta
#999 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub
Category: Owners P3: When Possible Type: Feature Request
Lingua principale
TypeScript
Stelle
35
Fork
35
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Direzione di ricerca

Inizia da owners/src/api/github.js#GitHub e owners/src/notifier.js#createNotificationComment, poi esamina il commento collegato della pull request sul comportamento esistente delle notifiche. Il lavoro è completato quando la GitHub API può eliminare il commento esistente di Owners Bot quando l’insieme delle notifiche diventa vuoto, senza scambiare altri commenti dei bot per quello.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
github, typescript
Ambito
api, tooling
Tipo di issue
Funzionalità
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Abbastanza chiara
Idoneità per principianti
45/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.