5 consejos para revisiones de código

Las revisiones de código son uno de los puntos principales a la hora de realizar código de calidad. En ella, se pretende asegurar el cumplimiento de los estándares de programación, ya sean organizativos, de equipo, o del propio lenguaje de programación. Por tanto, pueden participar tanto miembros del equipo como de la organización. Esto hace que las revisiones de código puedan ser un foco de discusión en el cual hay que ser lo suficientemente cuidadoso para no generar conflictos.

Os voy a recoger lo que para mí son los 5 puntos principales para realizar revisiones de código rápidas, fiables y que no generen controversia.

1. Entiendo, luego comento

Antes de empezar a revisar el código, conviene comprender el contexto de la Pull Request. Evitarás comentarios innecesarios que en la mayoría de casos se resuelven leyendo el funcional.

2. No impongas tu visión

Abre espacio al diálogo e invita a la otra persona a ver tu punto de vista. Recuerda que estás en un foro de discusión.

3. Define puntos de criticidad

Establece junto a tu equipo una serie de reglas a la hora de revisar código y define una prioridad. Tiene más criticidad comentar un posible bug que comentar un refactor de naming de variables. A menos tiempo para que el código a revisar tenga que estar en producción, más foco en los puntos críticos.

4. Regla de los 2 párrafos

Si en algún comentario recibes una respuesta de 2 o más párrafos, por lo general no vas a resolver el comentario en la siguiente interacción. Es por tanto más óptimo juntarse, realizando una llamada o abrir un chat y llegar a un consenso.

5. Discusión a 3

En el caso de encontrarte en una discusión en el cual ambos puntos de vista sean dispares, busca a una tercera persona que pueda intermediar en la discusión. Recuerda que el objetivo es no invertir más que el tiempo justo y necesario.