Quando verranno uniti i PR?

Abbastanza spesso, e devo dire abbastanza velocemente, quando qualcuno segnala un bug o un problema, qualcuno del team dice questa PR lo risolve.

E subito dopo l’utente ansioso effettua una ricompilazione o un aggiornamento, torna indietro e piange per chiedere aiuto perché quella correzione non ha cambiato nulla.

Lo stesso povero membro del team risponde che la PR non è ancora stata unita.

Qual è il lasso di tempo tipico per l’unione, o dipende? Molto spesso dite che la PR è già stata unita, o c’è una nuova risposta per informare dell’unione — ma non sempre.

Non ho problemi, sono solo puramente curioso. Beh… a volte è importante se una PR risolverà qualcosa di importante per me o mi darà qualcosa di bello, e ottenerlo richiede una ricompilazione, perché richiede molto tempo. Ma devo dire — ho ottenuto tutto ciò che è stato segnalato come PR.

Suppongo che il team debba fare qualche doppio controllo prima di unire qualcosa.

Temo che la risposta sia: “dipende”. Come minimo, dobbiamo attendere il completamento delle verifiche automatiche (circa 20 minuti) e abbiamo bisogno che almeno un’altra persona del Team esamini/approvi la PR. Questo può richiedere da pochi minuti a giorni, a seconda della disponibilità/priorità.

Se si tratta di una modifica più complessa, o se il revisore ha commenti da affrontare, allora può richiedere ancora più tempo.

A proposito, quando colleghiamo una PR qui su Meta, dovresti vedere un piccolo badge colorato sulla onebox che ti dice se è stata unita. Ad esempio:

10 Mi Piace

Certo, quel badge :man_facepalming:

Sì, dipende è una buona risposta. E anche una risposta attesa.

3 Mi Piace