envoyproxy/envoy

RepoKitteh-based PR checklists

Open

#12.949 aperta il 2 set 2020

Vedi su GitHub
 (13 commenti) (0 reazioni) (1 assegnatario)C++ (5373 fork)batch import
area/repokittehbeginnerenhancementhelp wanted

Metriche repository

Star
 (27.997 star)
Metriche merge PR
 (Merge medio 8g) (378 PR mergiate in 30 g)

Descrizione

We can have RK automatically label any change to source/ or api/ without a change to current.rst with a release-notes blocker, requiring manual confirmation from maintainers that the release notes step has been satisfied. Similarly this could work for documentation checks, i.e. validate that at least some change has happened in docs/.

I'd propose that we take the existing ownerscheck.star and add a path expression that suppresses a given check. That should be all that is needed for release notes or docs.

@envoyproxy/maintainers are there other common review checklist items that RK could help with? CC @itayd

Guida contributor

Direzione di ricerca
Esamina il file esistente ownerscheck.star nel repository, comprendi come funzionano le espressioni di percorso e aggiungi una nuova soppressione di percorso per le note di rilascio e i controlli della documentazione.
Tech stack
Nessuno
Dominio
devops
Tipo issue
Funzionalità
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
Git
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65