Come eseguire la revisione del codice con TFS?

Eventuali duplicati:
Web Based Code-Review tools for Team Foundation Server Come eseguire la revisione del codice con TFS?

Abbiamo circa 4 sviluppatori e stiamo utilizzando TFS per il controllo di origine.

Vorremmo introdurre la revisione del codice in modo che i codici archiviati siano esaminati da membri pari per prestazioni, design e standard.

Quale strumento consiglieresti di utilizzare per aiutare questo? (open source e gratis speriamo)

Ho trovato questo da Google: http://teamreview.codeplex.com/ e sto provando a verificare come funziona.

Grazie,

fonte

2012-01-05 The Light

http://stackoverflow.com/questions/1235410/web-based-code-review-tools-for-team-foundation-serverPossibile duplicato? – tafa

TeamReview era esattamente ciò di cui avevamo bisogno – pantelif

Non so di alcun attrezzo, di per sé, ma quando si utilizza TFS in un piccolo team, ho scoperto che semplicemente il backup delle modifiche su un shelveset era adeguata. In questo modo, altri sviluppatori hanno potuto visualizzare gli scaffali degli altri e passare attraverso le differenze di codice nel loro tempo. Possono anche annullare le modifiche sulle proprie macchine se desiderano verificare la compilazione e l'esecuzione del codice.

Qualcos'altro che è possibile fare per aiutare a verificare che il codice sia corretto, sta scrivendo unit test e sottoponendoli a revisione del codice. In questo modo, il sistema di test può aiutarti a identificare potenziali difetti di comportamento che potrebbero essere sfuggiti agli occhi di un revisore del codice.

fonte

2012-01-05 09:55:12

grazie, sì, questo aiuterebbe, ma il nostro team è attualmente contrario a scrivere test di unità per alcune ragioni! quindi non possiamo adottare questo approccio. Il revisore del codice dovrebbe essere solo un esperto di tecnologia o tutti i membri del team? –

Probabilmente è meglio essere tutti membri della squadra. Forse non tutti i membri su un pezzo di codice (a meno che non sia molto significativo). Ma assicurati che tutti i membri prendano il tempo di rivedere il codice di ogni sviluppatore ad un certo punto. Ciò contribuirà a mantenere la coerenza con gli standard di codifica e il team otterrà una migliore comprensione di una porzione più ampia della base di codice. E per la cronaca, i test unitari sono una cosa meravigliosa e, anche se potrebbe richiedere del tempo per iniziare, è possibile risparmiare ore di debugging e molti soldi dell'azienda a lungo termine! –

Non sono sicuro di come questo si possa applicare a voi, ma ecco qui. Ho modificato il modello del processo WIT (utilizzando il TFS power tools) in modo che lo stato dopo il check-in del codice sia "In attesa di revisione del codice". È quindi necessario assegnare questo WI a un altro utente che lo sposta nello stato "Chiuso" se la revisione del codice è OK, o torna a assegnato se è fallito.

Così le transizioni di processo sono ora qualcosa di simile:

 
Assigned --Check-In-->Code Review --Success-->Closed 
    ^-------Failed---------|

Questo è ben lungi dall'essere perfetto, ed è in gran parte sulla base dei processi che abbiamo avuto con Subversion ed un sistema di bug tracking indipendente, ma funziona.

In altre notizie, VS 11 sembra avere questo out of the box (anche here - preso dalla risposta di Ewald).

fonte

2012-01-05 10:10:51 DaveShaw

+1 - Questo è un buon approccio. E mi piace il fatto che VS11 sembra averlo già integrato. –

+1, sembra interessante. Mi sembra anche di avere il seguente svantaggio: tutte le tue WI sono chiuse dai revisori. Quindi, come puoi conoscere il carico di lavoro di ogni sviluppatore? – pantelif

@pantelif, Siamo ancora nuovi in TFS, quindi non abbiamo ancora attraversato quel ponte. – DaveShaw

Non ti aiuterà in questo momento, ma Dev11 (che è disponibile attualmente come anteprima per sviluppatori) include la revisione del codice. Puoi vedere una bella panoramica allo session from Jamie Cool at //BUILD.

Quindi, prima di prendere decisioni importanti, potresti voler aspettare un paio di mesi prima di poterlo utilizzare nel tuo lavoro quotidiano.

Ewald Hofman - Program Manager TFS

fonte

2012-01-05 16:18:53

posso raccomandare Review Assistant code review tool. Si integra in Visual Studio 2010 e 2012 e funziona con TFS. Come posso vedere dal tuo post, il processo di revisione del codice post-commit è preferibile per te. Questo preciso modello è implementato nello strumento.

fonte

2012-08-21 12:25:58 Devart

Come eseguire la revisione del codice con TFS?

risposta

Problemi correlati