Don't wait for PR approval when PR author is a maintainer
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 52/100
Direzione di ricerca
Leggi ci-bench-runner/src/job/bench_pr.rs, in particolare il controllo esistente dell’associazione dell’account consentita alle righe 29-36, quindi segui l’elaborazione degli eventi per un nuovo PR. La modifica è completa quando i PR creati da un maintainer di un’organizzazione non richiedono più approvazione o un commento prima del benchmarking, mentre gli altri PR mantengono il comportamento attuale.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
An idea from djc in discord:
would be nice if the benchmarking bot knew it doesn't need to wait for approval on PRs where the author is an org maintainer
I think this would be a fairly small lift: we already have the code in place to check when a PR is approved that the approver has an allowed account association. Likely we just need to adjust the event processing for a new PR to skip requiring an approval or a comment if the author has such an association.
- Lingua principale
- Rust
- Stelle
- 13
- Fork
- 3
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di rustls/rustls-bench-app
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 45/100
rustls/rustls-bench-app#34 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
rustls/rustls-bench-app#33 · 2 commenti ·
Tutte le issue di rustls/rustls-bench-app
Issue simili
-
bug priority: medium
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
storytold/photocraft#12706 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
IngvarConsulting/unica#2822 ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
-
Clean master build fails in zune-jpeg error conversion and zero-IV array construction (patch)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100