remota - ¿Cómo configurar una revisión de código usando Gitlab?
git push example (4)
He estado haciendo revisiones de código en Gitlab durante más de dos meses casi sin fricción. Configuré rss2email para enviar notificaciones por correo electrónico cada vez que un desarrollador envía nuevas confirmaciones. Luego uso la función de comentarios de Gitlab para los commits para hacer algunos comentarios sobre el código insertado.
Desafortunadamente, Gitlab no permite comentarios en los archivos en sí, solo en commits (al igual que Github, supongo). Cada vez que me encuentro en una situación en la que tengo que comentar algo que he omitido en una confirmación anterior, utilizo la herramienta de culpa para encontrar el comentario que introdujo / modificó la sección del código.
Está lejos de ser perfecto, pero está funcionando bien hasta ahora.
¿Cómo se configura una revisión de código usando Gitlab? Lo veo como una característica en el sitio web de Gitlab, pero parece que no puedo encontrar instrucciones sobre cómo configurar uno (para ese asunto, cualquier enlace a un manual de usuario de Gitlab sería muy apreciado).
Algunas de mis búsquedas indicaron que ''Merge Requests'' es el camino a seguir ... pero los estoy encontrando limitantes. Una solicitud de fusión emitida muestra todas las confirmaciones entre una rama y la otra. Parece que solo puedo ver las diferencias generadas para cada compromiso individual. Por ejemplo, digamos que tengo un archivo que quiero revisar. Es un archivo nuevo pero le envié cambios en más de 10 confirmaciones en una rama de desarrollo. Si emite una solicitud de fusión para esa rama dev desde la integración, veo 10 confirmaciones, cada una de las cuales muestra los cambios incrementales realizados en el archivo ... Quiero revisar todo el asunto. ¡Es nuevo!
¿Estoy ladrando el árbol equivocado aquí? ¿Existe alguna herramienta de revisión de código que pueda usar en GitLab, o las solicitudes de fusión son el camino a seguir, y si lo están, las estoy usando incorrectamente? ¿Cuál es la mejor manera de configurar una revisión de código adecuada aquí?
Nota: desde GitLab 6.4, vista de diff uno al lado del otro está disponible: ver " pull request 5308 ".
(Julio 2013) Sin embargo, todavía no hay posibilidad de comentar cada línea, solo a nivel de archivo.
Daniel Sokolowski menciona en los comentarios que los comentarios por línea ahora son compatibles (09/2014):
Los miembros de su equipo pueden comentar la solicitud de fusión en general o en líneas específicas con comentarios de línea.
Eso todavía puede ayudar con la actividad de revisión del código.
Puede ver el código enviado en Merge Request para otro repositorio o en el repositorio actual.
ejemplo http://demo.gitlab.com/diaspora/diaspora/commits/master
A continuación, puede agregar comentarios a los cambios de archivos confirmados (botón Responder ) o a toda la confirmación
ejemplo http://demo.gitlab.com/diaspora/diaspora/commit/42f47626890218a180870bc3f44ec57625b0779c
La comunicación resultante es revisión de código . Sin embargo, yo personalmente recomiendo revisar el código en una PC con comunicación cara a cara siempre que sea posible, y usar herramientas para registrar resultados o cuando se necesita más formalidad.
Para una revista de archivos que tiene muchas confirmaciones, p. Ej., http://demo.gitlab.com/diaspora/diaspora/blame/master/README.md mírelo con culpa para comprender quién hizo qué. Sin embargo, en esta vista no hay opción para comunicarse y agregar comentarios. Yo recomendaría simplemente agregar cambios como comentarios en este caso.
Sí. Las solicitudes de combinación son cómo se realizan las revisiones entre pares.
Debería haber una pestaña ''diff'' que mostrará los cambios de todos los commits (mencionados aquí: http://youtu.be/DyAX8ws5OIc?t=3m2s ).
El video también explica muy bien cómo se puede utilizar para la revisión por pares.