Come riesaminare una richiesta di pull?

4

La revisione di una richiesta di pull comprende test, revisione del codice o entrambi e quanto dovrebbe durare la media delle pubbliche relazioni?

Non so se il test richiede solo testare la funzionalità che è stata alterata, o testare completamente l'intera app e quale scopo ha questo se l'autore lo ha già testato.

Se si tratta di una revisione del codice dovrebbe essere una breve occhiata all'interno di github o una revisione approfondita che includa un'analisi tramite il codice in azione all'interno dell'IDE scelto.

Sono parte di un team di sviluppo a pagamento in cui il 100% dei nostri PR proviene dal nostro team.

    
posta Declan McKenna 10.08.2016 - 16:45
fonte

1 risposta

8

Ci sono compiti procedurali da eseguire in una revisione del codice:

  • Che il codice rispetti gli standard di codifica concordati dal team.
  • Che è completo per test unitari, documentazione, ecc.
  • Altri nella stessa vena.

Ma le attività più importanti in qualsiasi revisione del codice sono per verificare che:

  • Puoi capire il codice (tutto) perché è scritto chiaramente.
  • Il codice è corretto nel senso di assenza di errori (bug).
  • Il codice è corretto nella gestione dei casi di confine previsti.
  • Il codice è corretto in quanto svolge l'attività desiderata.

Inoltre, se sei educato, puoi suggerire modi migliori (più chiari, più efficienti, basati su librerie) per implementare la funzionalità.

    
risposta data 10.08.2016 - 17:10
fonte

Leggi altre domande sui tag