2009-06-04 8 views
22

Estoy buscando un código de revisión de complementos para nuestra instalación de trac.Trac: complemento de revisión de código

He encontrado estos dos como el primer resultado de la consulta "trac code review" en Google

me estoy inclinando hacia PeerCodeReview plugin.

Solicitando a la comunidad SO información sobre estos complementos para ayudarme a seleccionar uno para nuestra instalación de trac.

Si conoce algún otro plugin, por favor, hágamelo saber también. :)

lo que estoy buscando en el plugin

  • Una manera para anotar código con comentarios.
  • Aprobar/Desaprobar; como un botón para informar que el código debe cambiar. tal vez se crea un error.
  • Una forma de asignar el código de revisión "Tarea" a una persona (s).

Se requiere la primera característica (supongo que ese es el punto); otros son opcionales Puedo hackear Trac para obtener algo similar que se ajuste a ese flujo de trabajo. ¡Ojalá! ;)

+1

También hay GvnTrac: http: //projects.matt-good.net/trac/gvntrac, estas aplicaciones en trac-hacks: http://trac-hacks.org/tags/%27codereview%27, y es posible que también quiera echarle un vistazo a esta entrada : http://trac.edgewall.org/ticket/2035. – RjOllos

+0

Vea también: https://github.com/Automattic/trac-code-comments-plugin que parece actual e interesante –

Respuesta

6

Estábamos en la misma situación.Al final optamos por instalar Review Board en paralelo a Trac. Si bien no es un complemento de Trac, es una herramienta excelente, y hasta ahora estamos muy contentos con él.

También tiene soporte básico de Trac, por lo que, por ejemplo, al revisar las correcciones de errores, obtiene enlaces automáticos al ticket de Trac.

+0

¿Podría entrar en más detalles sobre la compatibilidad con Trac que se proporciona? ¿Quiere decir que hay un proveedor de TracLink para que pueda consultar revisiones específicas en ReviewBoard usando un enlace de un boleto o la wiki? – RjOllos

9

En mi compañía, miramos brevemente el plugin "peerreview" en TracHacks, y estuvimos muy decepcionados con él.

Parece obvio para nosotros que un complemento de revisión de código supondría naturalmente que toda la confirmación de Subversion debería revisarse por código. Desafortunadamente, el plugin peerreview te obliga a identificar manualmente las líneas de código que deseas revisar. Ni siquiera te da pistas sobre qué líneas podrían haber cambiado con una confirmación en particular, lo que significa que si no ingresas la línea que cambió cuidadosamente, podrías terminar re-revisando las mismas líneas de código una y otra vez.

No hemos tenido la oportunidad de revisar el otro complemento (CodeReviewPlugin) en detalle.

+0

Gracias por la información Eric :) – xk0der

4

Creo que PeerCodeReview es mejor para lo que quieres.

El póster anterior es correcto, sin embargo, debe buscar en el repositorio de origen y seleccionar el código que desea revisar y anotar. No sabe nada sobre los conjuntos de cambios.

Esto está bien si tienes a alguien (como un arquitecto o un cliente potencial) que está mirando proactivamente el código e identificando cosas que pueden ser problemáticas.

No funciona tan bien si desea revisar cada fragmento de código en lo que se refiere a un cambio.

Si usted está buscando algo que se asocia con confirmaciones, y se ocupa de las diferenciaciones, y luego mirar ReviewBoard (a DJango app):

+0

Gracias por su respuesta :) Echaré un vistazo a ReviewBoard. – xk0der

+0

¿Por qué recomienda PeerCodeReview sobre CodeReview? CodeReview no parece tener las deficiencias que describe para PeerCodeReview. Necesita algún trabajo (como un verdadero puerto para Trac 0.11 y Genshi), pero en general funciona bastante bien. – RjOllos

3

También estoy decepcionado con PeerCodeReview. Ata comentarios a una revisión específica, y cuando "reenvía" una revisión más reciente para su revisión, está cerrando la revisión anterior y abriendo una nueva, ¡pero todos los comentarios se quedan en la revisión anterior solamente! Por lo tanto, no hay una manera fácil de ver un comentario junto con la fuente más reciente que lo aborda. Junto con la falta total de soporte para revisar commits/diffs, esto hace que PeerCodeReview no sea adecuado para mí.

El plugin de CodeReview parece agradable (en realidad no lo intenté), pero todavía echo de menos la posibilidad de comentar líneas específicas.

No debería ser que mi caso de uso no fuera una "revisión de código" clásica, sino que revisara un único documento LaTeX. Esto tiene diferentes requisitos:

  • No es necesario revisar antes de una confirmación; por el contrario, un flujo de trabajo "pipeline" es crítico: los comentarios de revisión se crean cuando el revisor tiene tiempo libre, y se dirigen cuando el escritor los recibe, con frecuencia en confirmaciones posteriores.

  • Sería bueno hacer un seguimiento de qué partes del texto se han revisado en qué revisiones.
    Esto no es crítico, ya que la revisión generalmente se realiza una sección a la vez y es fácil suficiente para rastrear manualmente.

  • Hay muchos pequeños comentarios independientes. Cada comentario debe rastrearse de forma independiente, una única resolución de "aprobación/rechazo" para una confirmación.

  • La mayoría de los comentarios son bien localizadas en el archivo, por lo que quieren un flujo que permite unir a lugares específicos en el archivo, y ellos deben tener su lugar cuando el archivo es editado.

El derecho flujo para esto sería abrir una entrada para cada comentario, cerrándolos con cometer ganchos cuando se le habla. El único problema es que no hay una manera fácil de vincular un ticket a líneas específicas en el archivo, lo que lo hace bastante engorroso. Estoy tentado de escribir un complemento minimalista que vincule los tickets a las líneas fuente/diff. ¿Alguien más le gustaría tal bestia?

Lo que probablemente haré en la práctica es poner los comentarios de TODO dentro de la propia fuente y no utilizar la interfaz de Trac de lujo en absoluto para esto. El control de versiones se asegurará de que los comentarios permanezcan donde pertenecen entre ediciones.

[específicamente para el látex, probablemente usará los todonotes y/o fixme paquetes para mostrar los comentarios muy bien, y tal vez latexdiff para ficheros diff visuales.] Voy a editar esta tarde para informar de cómo ha ido ...

Por cierto, este enfoque no se limita a los documentos: he trabajado con un equipo que lo usó mucho para la revisión de código durante el desarrollo ágil intensivo y funcionó bastante bien.Tenían un deseo similar de "canalizar" el proceso de revisión: el desarrollo debe continuar, pero todos los cambios deben revisarse antes de hacer una publicación. La parte más difícil fue rastrear lo que se revisó o no, lo que se hizo marcando revisiones "limpias" y diferenciándolas; en retrospectiva, tener una rama "revisada" y "cherry-picking" en ella funcionaría mejor. (Por supuesto, las preguntas arquitectónicas no localizadas entrarían en las entradas de Trac, o comenzarían como comentarios de HACER y se migrarían a las entradas si resultaran no triviales)

Cuestiones relacionadas