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

127

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?

John
la source
+1 pour "Y a-t-il des sites qui affichent les meilleures pratiques". Les ALM Rangers sont généralement mes "goto guys" pour une utilisation approfondie, mais les seules informations que je peux trouver ne répondent pas à votre question. J'aimerais voir quelques idées et scénarios d'utilisation de la révision du code.
DaveShaw

Réponses:

164

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:

entrez la description de l'image 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:

entrez la description de l'image ici


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

entrez la description de l'image ici


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

entrez la description de l'image ici

Chad Dienhart
la source
7
Le problème avec cette solution est qu'elle est destructrice en ce que tous les commentaires sont époustouflés. Je pense qu'une meilleure solution serait de créer une nouvelle révision de code et de définir l'ancienne comme élément connexe.
Bardia
15
Le remplacement du plateau ne touche pas les commentaires uniquement les fichiers. Notre équipe utilise cette méthode avec succès et nous ne perdons aucun commentaire.
Chad Dienhart
10
Cela fonctionne, mais si vous ajoutez / supprimez des lignes lorsque vous modifiez un fichier et remplacez l'étagère, les commentaires qui figuraient sur ce fichier pointeront vers les mauvaises lignes.
Chin
2
Je n'aime généralement pas répéter, mais le point de @ Chin ci-dessus rend cette réponse complètement intenable. Oui, vous pouvez facilement mettre à jour votre étagère, mais ce faisant, TOUS les commentaires des critiques précédentes pointent désormais vers un texte incorrect. (Parce qu'apparemment, un commentaire de révision est attaché à un décalage absolu depuis le début d'un fichier.)
Michael Sorens
7
Trop manuel et pas la chose idéale. TFS ou Visual Studio devrait avoir quelque chose pour comparer les modifications entre les révisions consécutives, afin que le réviseur puisse voir exactement les modifications apportées depuis la dernière révision. N'existe-t-il rien de tel? Peut-être un plugin ou quelque chose ...
Iúri dos Anjos
6

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.

Paddy
la source
2

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.

Julealgon
la source
1
Génial. J'attendrai cela avec impatience aussi. J'apprécierais une vraie solution. Mon message est simplement un moyen de contourner le problème.
Chad Dienhart
Espérons que cela ne ressemblera pas à des pull requests azure DevOps. C'est un cauchemar de spam.
StingyJack
0

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.

Ville Pelho
la source