web-dev-qa-db-fra.com

Examens du code TFS - Afficher les fichiers mis à jour en réponse aux commentaires

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

Donc, 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.

Comment la personne 1 montre-t-elle maintenant ces modifications à 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éé lors de la demande de révision d'origine. La personne 1 doit-elle clôturer cet avis et en demander un deuxième?

La création d'un deuxième avis ne semble pas optimale, car vous perdez l'historique de votre conversation sur les raisons pour lesquelles vous apportez des modifications.

De nombreux sites Web expliquent comment utiliser les fonctionnalités de base des revues de code, mais existe-t-il des sites présentant les meilleures pratiques? Par exemple, qui devrait cliquer sur les cases à cocher en regard des fichiers?

c#visual-studio-2012tfstfs2012tfs-code-review
118
John

Donc, 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 plateau associé à la révision du code
  5. La personne 1 ajoute des commentaires pour continuer la discussion
  6. Répétez les étapes 2 à 5 jusqu'à l'acceptation

Voici les étapes nécessaires à la mise à jour du plateau associé à la révision.

  1. Dans le volet "Révision du code", sélectionnez le lien "Afficher le plateau".
  2. Dans le volet "Détails du plateau", mettez en surbrillance et copiez le nom du plateau
  3. Accédez au volet "Modifications en attente", cliquez sur "Shelve" et collez le nom du plateau.
  4. Appuyez sur le bouton Oui de la boîte de dialogue Remplacer la vérification du plateau
  5. Le réviseur peut maintenant voir les fichiers mis à jour et la discussion de révision peut continuer

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

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

enter image description here

2) Dans le volet "Détails du plateau", mettez en surbrillance et copiez le nom du plateau comme indiqué ci-dessous:

enter image description here

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

enter image description here

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

enter image description here

150
Chad Dienhart

Je crois que la procédure correcte consiste à ce que la Personne 1 apporte les modifications et demande une nouvelle révision. Lorsque votre code doit fonctionner, cela signifie que vous allez le modifier. Vous souhaiterez donc utiliser l'ancienne version à des fins de comparaison. Vous avez toujours la vieille critique dans l'historique après sa fermeture si vous souhaitez consulter les commentaires. Nous sommes en train d'optimiser notre processus de révision du code sur mon lieu de travail.

6
Paddy

Je sais que cette question est ancienne, mais elle n’est toujours pas corroborée, 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 ne très ancienne requête d’utilisation pour permettre la mise à jour d’une révision de code qui devrait résoudre ce problème avec élégance en faisant en sorte que les révisions de code aient des itérations.

2
julealgon

Vous devez le faire avec deux critiques différentes. Mais il y a aussi un moyen de garder l'historique avec le deuxième examen. Tout ce dont vous avez besoin sont des tâches.

Ce flux de travail est décrit pour les révisions basées sur des ensembles de modifications, mais il fonctionne également pour les révisions basées sur des modules.

  1. Créer tâche1
  2. Avant d’archiver changeset1, ajoutez task1 en tant qu’élément de travail associé.
  3. Enregistrement des modifications avec un élément de travail lié et demande de révision à cet ensemble de modifications
  4. Créer une tâche2
  5. Avant de vérifier dans changeset2, ajoutez les deux tâches en tant qu'élément de travail associé.
  6. Enregistrement des modifications avec deux éléments de travail liés et demande de révision à cet ensemble de modifications

Désormais, dans la deuxième demande de révision, le réviseur est en mesure de rechercher des tâches connexes. Si le relecteur examine la tâche1, il voit la modification1 et la demande de révision avec ses commentaires. Donc, vous ne perdrez pas l'historique des conversations.

0
Ville Pelho