Quando eseguire la revisione del codice

15

Recentemente siamo passati a un processo di mischia e stiamo lavorando su compiti e user story all'interno degli sprint. Vorremmo fare spesso delle revisioni del codice per renderle meno scoraggianti. Stiamo pensando di eseguirli a livello di utente, ma non siamo sicuri di come suddividere il nostro codice per tener conto di ciò.

Usiamo VS e TFS 2010 e siamo una squadra di 6.

Al momento ci occupiamo di funzionalità, ma stiamo lavorando al passaggio alla branching per la mischia.

Al momento non utilizziamo scaffalette e non vogliamo implementarle se esistono altre tecniche disponibili.

Come si consiglia di implementare la revisione del codice per ogni story utente?

    
posta mcass20 08.03.2011 - 20:14
fonte

5 risposte

3

Dipende dalla natura delle storie degli utenti.

Può essere efficace creare un ramo per ogni storia utente, i progressi su diverse storie sono visibili, possono essere passati in giro se necessario, se le storie non sono completate nello sprint, quindi i progressi possono rimanere nel ramo per il prossimo sprint. Le revisioni finali possono quindi essere eseguite alla fine di un articolo utente nel ramo della storia d'uso e unite nel caso in cui il codice sia conforme allo standard.

Per lavorare nel modo in cui le storie devono essere finemente granulose per evitare attività di unione non gestibili alla fine di uno sprint. Piccole storie permetteranno un costante aggiornamento del ramo di sviluppo attraverso lo sprint che gli sviluppatori che lavorano su altre storie di utenti devono costantemente estrarre da (VCM di base).

Questo crea i costi generali del processo che devono creare e unire costantemente i rami, che in alcuni casi possono essere risolti con gli script di automazione, ma il team deve comunque essere molto a suo agio con il VCS.

Alla fine di uno sprint unisci il tuo ramo dev in integrazione / produzione ecc.

Ho anche lavorato in team in cui tutti lavorano su un ramo di sviluppo, al termine di una user story il codice viene inviato a quel ramo per la revisione e il test e se qualcuno spinge qualcosa che rompe la build di sviluppo devono ottenere il team birre dentro.

    
risposta data 08.03.2011 - 20:35
fonte
13

Il modo più efficace per esaminare il codice è quello di alzarsi in piedi, trovare qualcuno e chiedere loro di venire a discutere del codice appena sviluppato.

Non utilizzare uno strumento a meno che non sia possibile trovare qualcuno che riveda il codice localmente.

È possibile evitare completamente le revisioni del codice tramite l'associazione.

    
risposta data 08.03.2011 - 22:32
fonte
4

Tutti i membri del team sono locali? Se è così, basta chiedere a qualcuno di venire a dare un'occhiata prima che il codice sia archiviato. Non locale? Avvia il tuo programma di condivisione dello schermo preferito e chiama qualcuno. Io personalmente lo faccio spesso. A volte lo faccio solo per dire "Ehi, guarda cosa ho fatto!"

Preferisco di gran lunga questo stile di recensioni di codici ad hoc allo stile in cui qualcuno è in piedi e presenta il proprio codice al team. Le recensioni ad hoc possono darti molti (tutti?) Dei benefici dell'accoppiamento senza l'imbarazzo. Inoltre, il tuo "revisore" ha maggiori probabilità di porre domande e suggerire miglioramenti in un ambiente informale, uno contro uno.

    
risposta data 09.03.2011 - 00:54
fonte
1

Credo che la revisione del codice non sia una parte formale di SCRUM, tuttavia le revisioni sono una tattica indipendente per raggiungere la qualità e migliorare i tuoi progetti / team.

Quindi, dovresti usare SCRUM (o un'altra metodologia di sviluppo agile) per assicurare / migliorare la qualità di PROJECT e continuare a pianificare. Inoltre, una buona tattica consiste nel fare la revisione del prodotto (non il codice) in modo indipendente dal normale QA / attività di test. Se questa attività potrebbe essere svolta di fronte al tuo team / partner / clienti / pubblico, sarà meglio.

Dovresti usare le revisioni del codice (o altre specifiche) principalmente per migliorare il tuo TEAM, aspettandoti i risultati a medio / lungo termine. Ciò influenzerà i tuoi PROGETTI, ma a lungo termine come un prodotto del tuo miglioramento TEAM.

Quindi, per rispondere alla tua domanda, credo che stai provando a spingere troppo da SCRUM, e dovresti prendere in considerazione le revisioni solo così com'è.

    
risposta data 08.03.2011 - 22:07
fonte
0

Non è ovvio effettuare revisioni del codice prima di controllare il codice?

TFS non funziona come GIT, quindi ogni volta che controlli il codice in un ramo o nel tronco, è disponibile per tutti.

Ciò significa che la verifica dovrebbe avvenire al momento del check-in, pertanto le modifiche errate non vengono propagate alla copia di lavoro di tutti.

    
risposta data 08.03.2011 - 20:16
fonte

Leggi altre domande sui tag