Recensioni del codice TFS: mostra i file aggiornati in risposta ai commenti

Stiamo iniziando a utilizzare la funzionalità di revisione del codice integrata nell'anteprima di VS 2012 e VS 2013. Richiedere la recensione e aggiungere commenti sembra abbastanza semplice. Se qualcuno aggiunge commenti che richiedono la modifica del codice, in che modo il richiedente apporta queste modifiche e le mostra?

Quindi il processo scorre in questo modo:

1. La persona 1 richiede una revisione del codice.
2. La persona 2 aggiunge commenti e seleziona "Ha bisogno di lavoro".
3. La persona 1 apporta le modifiche necessarie.

In che modo la persona 1 ora mostra queste modifiche alla persona 2? Puoi aggiungere commenti e inviarli, ma i file non cambiano. Suppongo che i file provengano dal changeset creato quando è stata richiesta la revisione originale. La persona 1 dovrebbe chiudere questa recensione e richiedere una seconda recensione?

La creazione di una seconda recensione non sembra ottimale, perché perdi la cronologia della tua conversazione sul perché stavi apportando le modifiche.

Ci sono tonnellate di siti Web che mostrano come utilizzare le funzionalità di base delle recensioni del codice, ma ci sono siti che mostrano le migliori pratiche? Ad esempio, chi dovrebbe fare clic sulle caselle di controllo accanto ai file?

Quindi il processo scorre in questo modo:

1. La persona 1 richiede una revisione del codice.
2. La persona 2 aggiunge commenti e seleziona "Ha bisogno di lavoro".
3. La persona 1 apporta le modifiche necessarie.
4. Persona 1 Aggiorna lo shelfet associato alla revisione del codice
5. La persona 1 aggiunge commenti per continuare la discussione
6. Ripetere i passaggi da 2 a 5 fino a quando non viene accettato

Ecco i passaggi necessari per aggiornare lo scaffale associato alla recensione.

1. Dal riquadro "Revisione codice" selezionare il collegamento "Visualizza scaffale"
2. Dal riquadro "Dettagli scaffale" evidenziare e copiare il nome dello scaffale
3. Passa al riquadro "Modifiche in sospeso", fai clic su "Shelve" e incolla il nome dello scaffale
4. Premi il pulsante Sì sulla finestra di dialogo di verifica della sostituzione dello scaffale
5. Ora il revisore può vedere i file aggiornati e la discussione sulla revisione può continuare

Ho incluso alcune schermate poiché trovo che aiuti a chiarire le cose.

1) Dal riquadro "Revisione codice" selezionare il collegamento "Visualizza scaffale" come mostrato qui:

2) Dal riquadro "Dettagli scaffale" evidenziare e copiare il nome dello scaffale come mostrato qui:

3) Passa al riquadro "In attesa di modifiche", fai clic su "Shelve" e incolla il nome dello scaffale, ad esempio:

4) Premere il pulsante Sì sulla finestra di dialogo di verifica della sostituzione dello scaffale:

Credo che la procedura corretta sia la persona 1 per apportare le modifiche e richiedere un'altra revisione. Quando il tuo codice ha bisogno di lavoro, significa che lo cambierai, quindi vorrai avere la versione precedente a cui tornare per un confronto. Hai ancora la vecchia recensione nella cronologia dopo che è stata chiusa se desideri esaminare i commenti. Attualmente stiamo ottimizzando il nostro processo di revisione del codice sul mio posto di lavoro.

So che questa domanda è vecchia, ma non è ancora supportata come sottolineato da altri poster. La soluzione proposta dal Ciad funzionerà per alcune cose, ma avrà un comportamento strano per altre.

Di recente, il team TFS ha iniziato le fasi di pianificazione per una soluzione a una richiesta di servizi utente molto vecchia per consentire l'aggiornamento di una revisione del codice che dovrebbe risolvere elegantemente questo problema facendo in modo che le revisioni del codice abbiano iterazioni.

Devi farlo con due diverse recensioni. Ma c'è anche un modo per mantenere la storia con la seconda recensione. Tutto ciò che serve sono le attività.

Questo flusso di lavoro è descritto per le recensioni basate su changeset, ma funziona anche per le recensioni basate su shelve.

1. Crea task1
2. Prima di effettuare il check in changeset1 aggiungere task1 come elemento di lavoro correlato
3. Controlla le modifiche con un elemento di lavoro correlato e richiedi la revisione di questo gruppo di modifiche
4. Crea task2
5. Prima di effettuare il check in changeset2 aggiungere entrambe le attività come elemento di lavoro correlato.
6. Controlla le modifiche con due elementi di lavoro correlati e richiedi la revisione di questo gruppo di modifiche

Ora nella seconda richiesta di revisione il revisore è in grado di cercare attività correlate e se il revisore cerca l'attività1, vede il changeset1 e la richiesta di revisione con i suoi commenti. Quindi non perderai la cronologia delle conversazioni.