SerenityOS/serenity

PR Labeler: Does not properly relablel PRs

Open

#17.271 aperta il 2 feb 2023

Vedi su GitHub
 (0 commenti) (0 reazioni) (0 assegnatari)C++ (3328 fork)batch import
help wantedtask list

Metriche repository

Star
 (33.171 star)
Metriche merge PR
 (Merge medio 5g 11h) (35 PR mergiate in 30 g)

Descrizione

We added the handy PR labeler CI action, but it doesn't seem to add labels properly in the following cases:

  • When a review is submitted by someone other than the author, change the state from "needs review" to "waiting for author feedback"
  • When a review is approved, move the state to "community approved" or "maintainer approved" based on who did the approval.
  • When a PR is detected to have conflicts, add the "has conflicts" label
  • When changes are pushed, remove the "waiting for author feedback" label and add "needs review"

Guida contributor

Direzione di ricerca
Esamina il file YAML del workflow del PR labeler esistente (probabilmente in .github/workflows) e identifica perché non riesce a rietichettare correttamente i PR in caso di invio di revisione, approvazione, rilevamento conflitti e push. Quindi aggiungi i trigger di evento necessari e la logica condizionale per gestire ogni caso.
Tech stack
github actions
Dominio
ci cd
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
GitGitHub Actions
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70