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

Nous commençons à utiliser la fonctionnalité de révision de code intégrée à l'aperçu de VS 2012 et VS 2013. Demander la révision et ajouter des commentaires semble assez simple. Si quelqu'un ajoute des commentaires demandant que le code soit modifié, comment le demandeur effectue-t-il ces modifications et les affiche-t-il?

Ainsi, le processus se déroulerait comme ceci:

1. La personne 1 demande une révision du code.
2. La personne 2 ajoute des commentaires et sélectionne «Besoin de travail».
3. La personne 1 fait 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 proviennent du jeu de modifications créé lorsque la révision d'origine a été demandée. La personne 1 devrait-elle fermer cet avis et demander un deuxième examen?

Créer un deuxième avis ne semble pas optimal, car vous perdez l'historique de votre conversation sur les raisons pour lesquelles vous apportiez les modifications.

Il existe une multitude de sites Web montrant comment utiliser les fonctionnalités de base des revues de code, mais y a-t-il des sites qui présentent les meilleures pratiques? Par exemple, qui devrait cocher les cases à côté des fichiers?

Ainsi, le processus se déroulerait comme ceci:

1. La personne 1 demande une révision du code.
2. La personne 2 ajoute des commentaires et sélectionne «Besoin de travail».
3. La personne 1 apporte les modifications nécessaires.
4. Personne 1 Met à jour le jeu d'étagères associé à la révision du code
5. La personne 1 ajoute des commentaires pour poursuivre la discussion
6. Répétez les étapes 2 à 5 jusqu'à ce que cela soit accepté

Voici les étapes nécessaires pour mettre à jour le jeu d'étagères associé à la revue.

1. Dans le volet "Révision du code", sélectionnez le lien "Afficher le plateau"
2. Dans le volet "Détails de l'étagère", mettez en surbrillance et copiez le nom de l'étagère
3. Accédez au volet "Modifications en attente", cliquez sur "Shelve" 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 du plateau
5. Le réviseur peut maintenant voir les fichiers mis à jour et la discussion de révision peut se poursuivre

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

1) Dans le volet "Révision du code", sélectionnez le lien "Afficher les étagères" comme indiqué ici:

2) Dans le volet «Détails de l'étagère», mettez en surbrillance et copiez le nom de l'étagère comme indiqué ici:

3) Accédez au volet "Modifications en attente", cliquez sur "Shelve" et collez le nom du plateau par exemple:

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

Je pense que la procédure correcte est que la personne 1 effectue les modifications et demande un autre examen. Lorsque votre code aura besoin de travail, cela signifie que vous le changerez, vous voudrez donc avoir l'ancienne version à consulter pour comparaison. Vous avez toujours l'ancien avis 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 de code sur mon lieu de travail.

Je sais que cette question est ancienne, mais elle n'est toujours pas prise en charge comme le soulignent d'autres affiches. La solution proposée par chad fonctionnera pour certaines choses mais aura un comportement étrange pour d'autres.

Récemment, l'équipe TFS a commencé les étapes de planification d'une solution à une très ancienne requête uservoice pour permettre la mise à jour d'une révision de code qui devrait résoudre élégamment ce problème en faisant des révisions de code des itérations.

Vous devez le faire avec deux avis différents. Mais il existe également un moyen de conserver l'historique avec la deuxième revue. 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 ensembles de modifications, mais il fonctionne également pour les révisions basées sur les mises à l'écart.

1. Créer la tâche1
2. Avant d'archiver le changeset1, ajoutez la tâche1 en tant qu'élément de travail associé
3. Archiver les modifications avec un élément de travail associé et demander la révision de cet ensemble de modifications
4. Créer la tâche2
5. Avant d'archiver le changeset2, ajoutez les deux tâches en tant qu'élément de travail associé.
6. Archiver les modifications avec deux éléments de travail associés et demander la révision de cet ensemble de modifications

Désormais, dans la deuxième demande de révision, le réviseur est en mesure de rechercher les tâches associées et, si le réviseur regarde la tâche1, il / elle voit l'ensemble de modifications1 et la demande de révision avec ses commentaires. Vous ne perdrez donc pas l'historique des conversations.