¿Qué debo notar mientras se revisa el código de pares?

Son las pocas cosas en las que puedes prestar atención.

  • ¿Falló la compilación para hacer el parche en la compilación?
  • ¿Este parche falla en la prueba unitaria?
  • ¿Está este código disminuyendo la cobertura del código de prueba de la unidad o aumentándola? Es posible que no desee aceptar la combinación si la cobertura del código de prueba de la unidad está fallando.
  • Asegúrese de que el análisis de código estático no se esté quejando de ningún riesgo potencial (Sonar, Findbugs, PMD, checkstytle, etc.).
  • En ellos han realizado el control de calidad para su código, luego verifique las pruebas realizadas en el mismo código y verifíquelo.

Las cosas anteriores pueden ser informadas por su sistema de CI, ahora es el momento para que usted muestre sus habilidades.

  • Revise el parche para ver si hay funciones como códigos, bucles, códigos, etc.
  • Revise el código línea por línea y muestre sus habilidades de codificación.