El Teatro de los Pull Requests y la Revisión de Código

La charla de Saša Jurić en Goatmire Elixir Conf transformó la revisión de código en una narrativa convincente. Destacó el problema común de los pull requests (PRs) demasiado grandes y complejos, lo que lleva a revisiones superficiales, riesgos de seguridad y bases de código difíciles de mantener. La conclusión principal: los PRs revisables deberían idealmente tomar de 5 a 10 minutos para revisarlos, idealmente con menos de 300 líneas de código. Esto se logra creando mensajes de commit concisos y narrativos que explican claramente el razonamiento y los pasos de cada cambio. Dividir las grandes funciones en PRs más pequeños y usar herramientas como `git fixup` para mantener un historial de commit limpio son cruciales para una revisión de código eficiente y un código de mayor calidad. La charla enfatizó que decir "No entiendo" es mejor que un "LGTM" sin sentido.