Cómo hacemos las revisiones de código
Nos enorgullece superar las expectativas de nuestros clientes con aplicaciones web y móviles avanzadas que encantan a sus usuarios. Las revisiones exhaustivas de código son una parte importante de cómo lo logramos, porque nos permiten comprobar que escribimos código bien estructurado y fácil de mantener.
Aunque practicamos la integración continua y ejecutamos pruebas automáticas con regularidad, la revisión humana sigue siendo fundamental para asegurar que nuestro código tenga alta calidad y funcione bien. Por eso las revisiones de código forman parte de nuestro proceso de desarrollo en sophilabs. No integramos código nuevo en la rama principal de desarrollo hasta que al menos otro integrante del equipo lo haya revisado y hayamos realizado los cambios necesarios. En este post explicaremos nuestro enfoque y recorreremos el proceso paso a paso.
Paso uno: crear una solicitud de integración
Cuando uno de nuestros desarrolladores termina de escribir el código de una funcionalidad, crea un pull request si usa GitHub o un merge request si usa GitLab, y asigna ese código a un compañero para que lo revise.
Lo ideal es pedir la revisión al compañero que escribió la versión original del código de esa funcionalidad. Si no es posible, se puede asignar a alguien del equipo que conozca bien el lenguaje de programación correspondiente.
Si la solución involucra dos lenguajes de programación, el desarrollador puede elegir a más de un revisor según los conocimientos y habilidades de sus compañeros. También puede pedir a más de una persona que revise su trabajo si la funcionalidad es especialmente crítica para el software o si los cambios son importantes o complejos.
Paso dos: recibir comentarios
En esta fase, el revisor examina el trabajo de su compañero y estudia detenidamente todos los cambios. Además de comprobar que el código sea correcto, prueba la funcionalidad para verificar que realmente funcione como se espera.
GitHub y GitLab permiten dejar comentarios escritos donde el revisor detecte un problema. El desarrollador puede responder a esos comentarios si tiene preguntas. Ambos también pueden reunirse brevemente en persona si hay cuestiones complejas que se explican mejor conversando.
En sophilabs, nuestra regla es concentrarnos en el trabajo y no en la persona, especialmente al dar comentarios constructivos. También vemos que cuanto más específico sea el revisor, más útiles resultan sus comentarios para el desarrollador.
Paso tres: hacer cambios y revisar
En esta fase, el desarrollador mejora el código. Después, el revisor vuelve a examinarlo y aprueba los cambios o aporta más comentarios si todavía hay aspectos que ajustar. Este proceso colaborativo puede tener varias rondas, hasta que el código esté en excelentes condiciones y el revisor apruebe el pull request.
Nuestros estándares de código son rigurosos, y el esfuerzo conjunto durante la revisión mejora su calidad. También trabajamos en equipo, por lo que a menudo nos ofrecemos a ayudar a implementar cambios, especialmente cuando un problema es complicado.
Paso cuatro: integrar el código
La última etapa consiste en integrar el código en la rama principal de desarrollo. En sophilabs asumimos la responsabilidad de nuestro trabajo: aunque recibimos ayuda y comentarios de nuestros compañeros, consideramos que el buen funcionamiento de nuestras contribuciones individuales al software es, en última instancia, nuestra responsabilidad personal.
Crear software de alta calidad es una tarea compleja. Incluir revisiones de código como parte obligatoria del proceso de desarrollo es una de las formas de asegurarnos de que creamos un producto excelente. Creemos que las perspectivas diversas mejoran las soluciones y nos exigimos mutuamente hacer nuestro mejor trabajo.
En sophilabs, nuestro principal propósito es desarrollar software que responda a las necesidades de nuestros clientes, por lo que diseñamos prácticas que aseguren su éxito. Si quieres saber más sobre cómo trabajamos, consulta nuestro proceso de desarrollo.