Cómo hacen code review distintas empresas de TI

• Razones para hacer code review:

  • Motivación: actuar con profesionalismo para no perjudicar al revisor
  • Compartir conocimiento: mejorar la capacidad técnica de toda la organización mediante el intercambio bidireccional de conocimiento
  • Detección de errores: aumenta la posibilidad de detectar errores de forma anticipada

• Google:

  • Todo el código
  • Siguiendo reglas alineadas con las políticas internas
  • Lo más pequeño posible y con frecuencia
  • Revisar dentro de 4 horas (reviews pequeñas dentro de 1 hora, reviews grandes dentro de 5 horas)

• Microsoft

  • El 39% hace reviews al menos una vez al día

• BankSalad

  • Hacen PR pequeños con frecuencia
  • Crearon plantillas con los cambios, pantallas relacionadas y checklist de pruebas para que el revisor entienda fácil y revise rápido

• Woowa Brothers

  • Crearon y usan una plantilla de MR
  • Usan un bot de MR para recibir alertas sobre "qué MR está llegando"

• Toss Payments

  • A veces agregan una etiqueta llamada Good Review para expresar comentarios positivos

• Wemakeprice

  • Recomiendan reviews detalladas y el uso de emoticonos

[kuber]

Microsoft: 39% de las personas revisan al menos una vez al día
...?
¿No es poquísimo? Me sorprende

[superwoou]

Parece que depende de cada empresa. En la nuestra, todos los ingenieros hacen revisiones al menos dos veces al día... jaja

[ironlung]

Esto suele interpretarse de forma distinta según quién lo vea. ^^;