2012-01-05 19 views
6

Duplicar posible:
Web Based Code-Review tools for Team Foundation Server¿Cómo hacer la revisión de código con TFS?

Tenemos alrededor de 4 desarrolladores y estamos utilizando TFS de control de código fuente.

Nos gustaría introducir la Revisión de códigos para que los códigos marcados sean revisados ​​por los miembros del equipo por su rendimiento, diseño y estándares.

¿Qué herramienta recomendaría que usáramos para ayudar a esto? (de código abierto y con suerte)

Encontré esto de Google: http://teamreview.codeplex.com/ y estoy tratando de probar cómo funciona.

Gracias,

+0

http://stackoverflow.com/questions/1235410/web-based-code-review-tools-for-team-foundation-serverPossible duplicate? – tafa

+0

TeamReview era todo lo que necesitábamos – pantelif

Respuesta

1

No sé de ninguna herramienta, per se, pero cuando se utiliza TFS en un equipo pequeño, me encontré con que simple apoyo a los cambios en una shelveset era adecuado. De esta manera, otros desarrolladores podrían ver los juegos de estanterías de otros y superar las diferencias de código en su propio tiempo. También pueden desmantelar los cambios en sus propias máquinas si desean verificar la compilación y ejecución del código.

Otra cosa que puede hacer para ayudar a asegurarse de que el código es correcto, es escribir pruebas de unidades y ponerlas bajo revisión de código. De esta forma, el sistema de prueba puede ayudarlo a identificar posibles defectos de comportamiento que el ojo de un revisor de códigos puede pasar por alto.

+0

gracias, sí, eso ayudaría, pero nuestro jefe de equipo actualmente se opone a las pruebas de unidades de escritura por alguna razón. entonces no podemos tomar ese enfoque. ¿El revisor de código debe ser solo un experto en tecnología o todos los miembros del equipo? –

+0

Probablemente sea mejor ser todos los miembros del equipo. Quizás no todos los miembros en una pieza de código (a menos que sea muy significativo). Pero asegúrese de que todos los miembros se tomen el tiempo para revisar el código de cada desarrollador en algún momento. Esto ayudará a mantener la coherencia con sus estándares de codificación y el equipo obtendrá una mejor comprensión de una porción más grande de la base de códigos. Y para el récord: la prueba unitaria es algo maravilloso y, aunque puede tomar tiempo configurarla al principio, ¡puede ahorrar horas de depuración y mucho dinero de la compañía a largo plazo! –

8

No estoy seguro de cómo esto se aplicará a usted, pero aquí va. Cambié nuestra plantilla de proceso WIT (usando el TFS power tools) para que el estado después del registro del código sea "En espera de la revisión del código". Luego debe asignar este WI a otro usuario que lo mueva al estado "Cerrado" si la revisión del código es correcta, o volver a asignarla si falla.

Así las transiciones proceso son ahora algo como:

 
Assigned --Check-In-->Code Review --Success-->Closed 
    ^-------Failed---------| 

Esto está lejos de ser perfecto, y se basa en gran parte en los procesos que tuvimos con Subversion y un sistema de seguimiento de fallos por separado, pero funciona.

En otras noticias, VS 11 parece tener este out of the box (también here - tomado de la respuesta de Ewald).

+1

+1 - Es un buen enfoque. Y me gusta el hecho de que VS11 parece tenerlo ya integrado. –

+0

+1, parece interesante. Sin embargo, también me parece que tiene la siguiente desventaja: los revisores cierran todas sus WI. Entonces, ¿cómo puede saber la carga de trabajo de cada desarrollador? – pantelif

+0

@pantelif, Todavía somos nuevos en TFS, por lo que aún no hemos cruzado ese puente. – DaveShaw

1

No lo va a ayudar en este momento, pero Dev11 (que está disponible actualmente como una Vista previa del desarrollador) tiene incluida la revisión del código. Puede ver una descripción general agradable en session from Jamie Cool at //BUILD.

Así que antes de tomar decisiones importantes, es posible que desee esperar un par de meses antes de poder utilizar esto en su trabajo diario.

Ewald Hofman - Administrador de Programas TFS

1

puedo recomendar Review Assistant code review tool. Se integra en Visual Studio 2010 y 2012, y funciona con TFS. Como puedo ver en su publicación, el proceso de revisión del código posterior a la confirmación es preferible para usted. Este modelo preciso se implementa en la herramienta.

Cuestiones relacionadas