Ottenere feedback con le pull request

Le richieste pull supportano la revisione e l'unione del codice in un unico processo collaborativo. Dopo che uno sviluppatore aggiunge una funzionalità o una correzione di bug, crea una richiesta pull per avviare il processo di unione delle modifiche nel ramo upstream. Gli altri membri del team hanno quindi la possibilità di esaminare e approvare il codice prima che venga finalizzato. Usare le richieste pull per esaminare i lavori in corso e ricevere tempestivamente il feedback sulle modifiche. Ma non c'è alcun impegno a unire le modifiche. Un proprietario può abbandonare una pull request in qualsiasi momento.

Ottenere il codice esaminato

La revisione del codice eseguita come parte di una richiesta pull non è solo per trovare bug evidenti; questo è ciò che i test sono per. Una buona revisione del codice rileva problemi meno evidenti che potrebbero causare problemi costosi in un secondo momento.

Le revisioni del codice aiutano a proteggere il team da merge errati e build interrotte che minano la produttività del team. Le revisioni rilevano i problemi prima dell'unione, proteggendo rami importanti da modifiche indesiderate.

Le revisioni del codice incoraggiano e rafforzano anche la collaborazione e la comunicazione tra sviluppatori. E il team ottiene una cronologia chiara di tutte le modifiche apportate tra il ramo principale e i rami di funzionalità.

Sfrutta l'expertise condivisa e diffondi strategie di risoluzione dei problemi utilizzando un'ampia gamma di revisori nelle revisioni del codice. La diffusione di competenze e conoscenze rende il team più forte e più resiliente.

Inviare commenti e suggerimenti di grande utilità

Le revisioni di qualità elevata iniziano con feedback di qualità elevata. Le chiavi per ottenere un feedback ottimale in una richiesta pull sono:

• Far esaminare la pull request alle persone giuste.
• Assicurarsi che i revisori sappiano cosa fa il codice.
• Dare feedback attuabile e costruttivo.
• Rispondere ai commenti in modo tempestivo.

Quando si assegnano revisori a una pull request, assicuratevi di selezionare il giusto set di revisori. I revisori devono sapere come funziona il codice, ma includono anche sviluppatori che lavorano in altre aree in modo che possano condividere le proprie idee.

Fornire una descrizione chiara delle modifiche e fornire una compilazione del codice che include la correzione o la funzionalità in uso. I revisori devono fare uno sforzo per fornire commenti e suggerimenti sulle modifiche che non sono d'accordo. Identificare il problema e fornire suggerimenti specifici su cosa potrebbe essere fatto in modo diverso. Questo feedback ha finalità chiare ed è facile da comprendere per il proprietario della richiesta pull.

Il proprietario della richiesta pull deve rispondere ai commenti, accettare suggerimenti o spiegare perché rifiuta di applicarli. Alcuni suggerimenti sono validi, ma potrebbero non rientrare nell'ambito della richiesta pull. Prendere questi suggerimenti e creare nuovi elementi di lavoro e rami di funzionalità separati dalla richiesta pull per apportare tali modifiche.

Proteggere i rami con le politiche

Esistono alcuni rami critici in un repository che i team si aspettano siano sempre in buone condizioni, ad esempio il main ramo. Teams può richiedere richieste pull per apportare modifiche a questi rami con piattaforme come GitHub e Azure DevOps. Gli sviluppatori che eseguono il push delle modifiche direttamente nei rami protetti avranno i push rifiutati.

Aggiungere condizioni aggiuntive alle richieste di pull per garantire un livello superiore di qualità del codice nei rami chiave. Una compilazione pulita del codice unito e l'approvazione di più revisori sono alcuni requisiti aggiuntivi spesso impiegati per proteggere i rami chiave.

Ulteriori informazioni

GitHub include una documentazione completa su come proporre modifiche al lavoro con le richieste pull.

Altre informazioni su come fornire commenti e suggerimenti utili nelle revisioni del codice e sull'uso di modelli di richiesta pull per fornire indicazioni ai revisori. Azure DevOps offre anche un'esperienza avanzata di richiesta pull facile da usare e ridimensionare in base alle esigenze.