TFS Code Reviews - Afficher les fichiers mis à jour en réponse aux commentaires

Question

Nous commençons à utiliser la fonctionnalité de revue de code intégrée à VS 2012 et VS 2013 preview. La demande de révision et l'ajout de commentaires semblent assez simples. Si quelqu'un ajoute des commentaires demandant que le code soit modifié, alors comment le demandeur effectue-t-il ces modifications et les montre-t-il ?

Le processus serait donc le suivant :

1. La personne 1 demande un examen du code.
2. La personne 2 ajoute des commentaires et sélectionne "Besoin de travail".
3. La personne 1 effectue les changements nécessaires.

Comment la personne 1 montre-t-elle maintenant ces changements à la personne 2 ? Vous pouvez ajouter des commentaires et les envoyer, mais les fichiers ne changent pas. Je suppose que les fichiers sont issus du jeu de modifications créé lorsque la révision initiale a été demandée. La personne 1 doit-elle fermer cette révision et demander une deuxième révision ?

La création d'une deuxième révision ne semble pas optimale, car vous perdez l'historique de votre conversation sur les raisons pour lesquelles vous avez effectué les changements.

Il existe des tonnes de sites web montrant comment utiliser la fonctionnalité de base des revues de code, mais y a-t-il des sites qui montrent les meilleures pratiques ? Par exemple, qui doit cocher les cases à côté des fichiers ?

Answer 1

Le processus serait donc le suivant :

1. La personne 1 demande un examen du code.
2. La personne 2 ajoute des commentaires et sélectionne "Besoin de travail".
3. La personne 1 effectue les changements nécessaires.
4. La personne 1 met à jour le jeu de tablettes associé à la revue de code.
5. La personne 1 ajoute des commentaires pour poursuivre la discussion
6. Répétez les étapes 2 à 5 jusqu'à ce que vous ayez accepté

Voici les étapes nécessaires pour mettre à jour le jeu de tablettes associé à la revue.

1. Dans le volet "Examen du code", sélectionnez le lien "Afficher le jeu de tablettes".
2. Dans le volet "Shelveset Details", mettez en surbrillance et copiez le nom du jeu d'étagères.
3. Accédez au volet "Modifications en attente", cliquez sur "Étagère" et collez le nom de l'étagère.
4. Appuyez sur le bouton Oui de la boîte de dialogue de vérification du remplacement des tablettes.
5. Maintenant, le réviseur peut voir les fichiers mis à jour et la discussion sur la révision peut continuer.

J'ai inclus quelques captures d'écran car je trouve que cela aide à clarifier les choses.

---

1) Dans le volet "Code Review", sélectionnez le lien "view shelveset" comme indiqué ici :

---

2) Dans le volet "Shelveset Details", mettez en surbrillance et copiez le nom du Shelveset comme indiqué ici :

---

3) Naviguez vers le volet "Pending Changes", cliquez sur "Shelve" et collez le nom de l'étagère par exemple :

---

4) Appuyez sur le bouton Oui de la boîte de dialogue de vérification du remplacement des tablettes :

Answer 2

Je crois que la procédure correcte est que la première personne fasse les changements et demande une autre révision. Lorsque votre code a besoin d'être retravaillé, cela signifie que vous allez le modifier et que vous voudrez avoir l'ancienne version pour pouvoir la comparer. Vous avez toujours l'ancienne révision dans l'historique après sa fermeture si vous souhaitez consulter les commentaires. Nous sommes actuellement en train d'optimiser notre processus de révision du code sur mon lieu de travail.

Answer 3

Je sais que cette question est ancienne, mais elle n'est toujours pas prise en charge, comme l'ont souligné d'autres posteurs. La solution proposée par Chad fonctionnera pour certaines choses mais aura un comportement étrange pour d'autres.

Récemment, l'équipe TSF a commencé à planifier une solution pour une très ancienne demande de l'utilisateur pour permettre la mise à jour d'un examen de code maintenant migré vers developercommunity.visualstudio qui devrait résoudre élégamment ce problème en faisant en sorte que les revues de code aient des itérations.

Answer 4

Vous devez le faire avec deux examens différents. Mais il existe aussi un moyen de conserver l'historique avec le deuxième examen. Tout ce dont vous avez besoin, ce sont des tâches.

Ce flux de travail est décrit pour les révisions basées sur les changesets, mais il fonctionne également pour les révisions basées sur les shelves.

1. Créer la tâche 1
2. Avant de vérifier le changeset1, ajoutez la tâche1 comme un élément de travail lié.
3. Enregistrez les modifications avec un élément de travail lié et demandez la révision de cet ensemble de modifications.
4. Créer la tâche 2
5. Avant de vérifier le changeset2, ajoutez les deux tâches comme un élément de travail lié.
6. Enregistrez les changements avec deux éléments de travail liés et demandez la révision de ce jeu de changements.

Maintenant, dans la deuxième demande de révision, le réviseur est capable de rechercher les tâches liées et si le réviseur regarde la tâche 1, il voit le changeset1 et la demande de révision avec ses commentaires. Ainsi, vous ne perdrez pas l'historique de la conversation.