Revisão do código antes do controlo para a TFS 2013

Question

Revisão do código antes do controlo para a TFS 2013

Estou a tentar implementar um processo para que o gerente possa rever o código de todos os desenvolvedores antes que os desenvolvedores possam entrar no TFS 2013.

Existe algum processo para exigir uma revisão do código por um ser humano antes de ser adicionado a um projeto real?

Estou a usar o TFS 2013 com o Visual Studio 2013.

15

visual-studio-2013 tfs2013 checkin pre-commit

Author: Jeroen, 2014-10-28

Source

1 answers

score 18 · Accepted Answer

O TFS 2013 suporta isto fora da caixa, e é muito direto para a frente para usar.

O programador precisa de apresentar um pedido de revisão no team explorer:

abrir as alterações pendentes
no menu "Acções", escolha a opção Request Review
especifique o revisor, e envie.

O utilizador-alvo (no seu caso, o gestor) será notificado (se tiver configurado a configuração do E-mail no TFS), e também poderá ver a revisão pedidos Na equipa Explorer > o meu trabalho.

O revisor pode ver todas as alterações feitas pelo desenvolvedor, comentar sobre elas e enviar os comentários para o desenvolvedor. A coisa toda acontece dentro do Visual Studio, e todos os comentários são mantidos em TFS e podem ser relatados mais tarde.

Além disso, após a conclusão da revisão, o revisor pode especificar sua opinião sobre as mudanças como "parece bom", "com comentários" ou "precisa de trabalho".

Certifique-se que lê esta página do MSDN sobre presente: http://msdn.microsoft.com/en-us/library/hh474795.aspx

Também pode ver este vídeo no Canal 9.: http://channel9.msdn.com/Series/Visual-Studio-2012-Premium-and-Ultimate-Overview/Visual-Studio-Ultimate-2012-Using-Code-Review-to-Improve-Quality

Editar

Para evitar o check-in sem uma revisão, você pode usar uma política de check-in. Aqui está uma política de check-in personalizada que requer revisão de código antes check-in:

Https://visualstudiogallery.msdn.microsoft.com/c476b708-77a8-4065-b9d0-919ab688f078

Mas devo salientar que as Políticas de check-in não são mecanismos de segurança, pense nelas como um lembrete para o desenvolvedor enviar o código para o check-in. Os desenvolvedores podem facilmente sobrepô-los.

Se realmente precisar de impedir qualquer check-ins, deverá negar a permissão de commit para os programadores, e fazê-los enviar as suas alterações para o gestor usando um code-review (or shelve set). O gerente pode, então, desbloquear as alterações e commit após a revisão.