Revisões de código TFS - Mostrar arquivos atualizados em resposta a comentários

127

Estamos começando a usar a funcionalidade de revisão de código incorporada à visualização do VS 2012 e do VS 2013. Solicitar a revisão e adicionar comentários parece bastante direto. Se alguém adicionar comentários solicitando a alteração do código, como o solicitante faz essas alterações e as mostra?

Portanto, o processo fluiria assim:

  1. A pessoa 1 solicita uma revisão de código.
  2. A pessoa 2 adiciona comentários e seleciona "Precisa de trabalho".
  3. A pessoa 1 faz as alterações necessárias.

Como a Pessoa 1 agora mostra essas alterações na Pessoa 2? Você pode adicionar comentários e enviá-los, mas os arquivos não são alterados. Suponho que os arquivos sejam do conjunto de alterações criado quando a revisão original foi solicitada. A pessoa 1 deve fechar esta revisão e solicitar uma segunda revisão?

Criar uma segunda revisão não parece ideal, porque você perde o histórico de sua conversa sobre o motivo de fazer as alterações.

Existem vários sites que mostram como usar a funcionalidade básica das revisões de código, mas existem sites que mostram as melhores práticas? Por exemplo, quem deve clicar nas caixas de seleção ao lado dos arquivos?

John
fonte
+1 em "existem sites que mostram as práticas recomendadas". Os ALM Rangers costumam ser meus "goto guys" para uso aprofundado, mas a única informação que posso encontrar não responde à sua pergunta. Eu adoraria ver alguns exemplos e cenários para usar a Revisão de Código.
precisa saber é o seguinte

Respostas:

164

Portanto, o processo fluiria assim:

  1. A pessoa 1 solicita uma revisão de código.
  2. A pessoa 2 adiciona comentários e seleciona "Precisa de trabalho".
  3. A pessoa 1 faz as alterações necessárias.
  4. Pessoa 1 Atualiza o conjunto de prateleiras associado à revisão de código
  5. A pessoa 1 adiciona comentários para continuar a discussão
  6. Repita as etapas 2 a 5 até ser aceito

Aqui estão as etapas necessárias para atualizar o conjunto de prateleiras associado à revisão.

  1. No painel "Revisão de código", selecione o link "visualizar prateleira"
  2. No painel "Detalhes do conjunto de prateleiras", realce e copie o nome do conjunto de prateleiras
  3. Navegue até o painel "Alterações pendentes", clique em "Arquivar" e cole o nome do conjunto de prateleiras
  4. Pressione o botão Sim no diálogo prateleira substituir verificação
  5. Agora, o revisor pode ver os arquivos atualizados e a discussão sobre revisão pode continuar

Incluí algumas capturas de tela, pois acho que ajuda a esclarecer as coisas.


1) No painel "Revisão de código", selecione o link "visualizar prateleira", como mostrado aqui:

insira a descrição da imagem aqui


2) No painel "Detalhes do conjunto de prateleiras", realce e copie o nome do conjunto de prateleiras, conforme mostrado aqui:

insira a descrição da imagem aqui


3) Navegue até o painel "Alterações pendentes", clique em "Arquivar" e cole o nome do conjunto de prateleiras, por exemplo:

insira a descrição da imagem aqui


4) Pressione o botão Sim no diálogo de verificação de substituição de prateleira:

insira a descrição da imagem aqui

Chad Dienhart
fonte
7
O problema com esta solução é que é destrutivo, pois todos os comentários são surpreendidos. Penso que uma solução melhor seria criar uma nova revisão de código e definir a antiga como um item relacionado.
Bardia
15
A substituição do conjunto de prateleiras não toca nos comentários apenas nos arquivos. Nossa equipe usa esse método com sucesso e não perdemos nenhum comentário de revisão.
Chad Dienhart
10
Isso funciona, mas se você adicionar / remover linhas ao editar um arquivo e substituir o conjunto de prateleiras, os comentários contidos nesse arquivo apontarão para as linhas incorretas.
Chin
2
Normalmente não gosto de repetir, mas o argumento de @ Chin acima torna essa resposta completamente insustentável. Sim, você pode atualizar facilmente seu conjunto de prateleiras, mas isso faz com que TODOS os comentários de revisão anteriores agora aponte para texto ERRADO. (Porque aparentemente uma crítica comentário é anexado a um deslocamento absoluto desde o início de um arquivo.)
Michael Sorens
7
Manual demais e não a coisa ideal. O TFS ou o Visual Studio deve ter algo para comparar as alterações entre revisões consecutivas, para que o revisor possa ver exatamente as alterações feitas desde a última revisão. Não existe algo assim? Talvez um plug-in ou algo assim ...
Iúri dos Anjos
6

Acredito que o procedimento correto é a Pessoa 1 para fazer as alterações e solicitar outra revisão. Quando o seu código precisa funcionar, significa que você o mudará, portanto, você precisará da versão antiga para uma comparação. Você ainda tem a revisão antiga no histórico depois de fechada, se desejar examinar os comentários. Atualmente, estamos no processo de otimizar nosso processo de revisão de código no meu local de trabalho.

Paddy
fonte
2

Sei que essa pergunta é antiga, mas ainda não é suportada, conforme apontado por outros pôsteres. A solução proposta pelo chad funcionará para algumas coisas, mas terá um comportamento estranho para outras.

Recentemente, a equipe do TFS iniciou os estágios de planejamento de uma solução para uma solicitação de serviço muito antiga, para permitir a atualização de uma Revisão de Código, que deve resolver esse problema com elegância, fazendo com que as Revisões de Código tenham iterações.

julealgon
fonte
1
Ótimo. Também estarei ansioso por isso. Gostaria de receber uma solução real. Minha postagem é simplesmente uma maneira de solucionar o problema.
Chad Dienhart 24/10/2015
Espero que não pareça solicitações de recebimento do DevOps do Azure. Eles são um pesadelo com spam.
precisa saber é o seguinte
0

Você precisa fazer isso com duas análises diferentes. Mas há também uma maneira de manter a história com a segunda revisão. Tudo que você precisa são tarefas.

Esse fluxo de trabalho é descrito para revisões baseadas em conjunto de alterações, mas também funciona para revisões baseadas em arquivamento.

  1. Criar tarefa1
  2. Antes de fazer o check-in do changeset1, adicione task1 como um item de trabalho relacionado
  3. Faça o check-in de alterações com um item de trabalho relacionado e solicite a revisão deste conjunto de alterações
  4. Criar tarefa2
  5. Antes de fazer o check-in do changeset2, adicione as duas tarefas como um item de trabalho relacionado.
  6. Faça o check-in de alterações com dois itens de trabalho relacionados e solicite a revisão deste conjunto de alterações

Agora, na segunda solicitação de revisão, o revisor pode procurar tarefas relacionadas e, se o revisor examinar a tarefa1, verá o conjunto de alterações1 e a solicitação de revisão com seus comentários. Então você não estará perdendo o histórico de conversas.

Ville Pelho
fonte