La verdadera razón por la que tus PR se vuelven enormes
Una vez trabajé en una empresa que enviaba pull requests masivos por costumbre. Un PR podía quedarse abierto durante semanas. Revisarlo requería tener todo un subsistema en la cabeza. Los errores se acumulaban. Los plazos se incumplían. Al final, tuvimos que reconstruir una gran parte del sistema porque ya nadie podía modificarlo de forma segura.
Los ingenieros no eran malos. Eran inteligentes y trabajaban duro. Los PR crecieron tanto por una razón aburrida.
Nadie les enseñó cómo desglosar el trabajo.
A menudo tratamos los PR grandes como un problema de disciplina. Decimos: "Simplemente haz PR más pequeños". Actuamos como si la fuerza de voluntad fuera la única diferencia entre 1,500 cambios y 150 cambios.
No se trata de fuerza de voluntad. Desglosar un trabajo grande en fragmentos pequeños e independientes es una habilidad. A la mayoría de la gente nunca se le enseña. Cuando un ticket dice "añadir facturación", parece una única tarea. La parte difícil es ver dónde termina un PR y dónde empieza el siguiente.
Yo también solía enviar PR grandes. Pensaba que "terminado" significaba resolver todo el problema de una vez y enviarlo para revisión. Me tomó años aprender que lo pequeño es mejor.
Los PR pequeños lo cambiaron todo para mí:
- Los revisores detectan más errores. Un humano puede razonar sobre 200 cambios. No puede razonar sobre 2,000. Simplemente los revisan por encima y los aprueban.
- Las fusiones ocurren más rápido. Los PR dejan de quedarse estancados en la cola.
- Dejé de sentirme abrumado. Me concentré en una pieza a la vez.
- Me convertí en un mejor planificador. Debes entender la estructura de tu trabajo para poder desglosarlo.
Empecé a ver los sistemas como piezas de Lego. Son piezas pequeñas que encajan entre sí. Una vez que ves los ladrillos, reducir el trabajo se siente natural.
Mi equipo actual envía PR pequeños. Los resultados son claros:
- Nuestro tiempo promedio de fusión es de 1.5 días.
- Encontramos y corregimos errores rápidamente porque los cambios son legibles.
- Nuestros tiempos de entrega son constantes.
Desglosar el trabajo es una habilidad que debes entrenar. No puedes solucionar los PR grandes con una regla. Los solucionas enseñando a la gente a ver los ladrillos.
La IA hace que esta habilidad sea aún más vital.
En el pasado, escribir 2,000 líneas de código requería mucho esfuerzo. Esa fricción mantenía los PR más pequeños. La IA eliminó esa fricción. Puedes generar cambios masivos con un solo prompt.
El esfuerzo no desapareció. Simplemente se trasladó al revisor. El autor no paga nada, pero el revisor paga el precio completo.
Si el tamaño ya no indica cuánto trabajo realizó un autor, el tamaño te dice muy poco sobre el riesgo. Debes decidir qué partes merecen tu atención más cuidadosa.
Enseña a tu equipo a ver los ladrillos. Es el hábito de mayor apalancamiento en ingeniería.
¿Cómo decide tu equipo qué PR necesitan una revisión profunda y cuáles pueden pasar rápidamente?
Fuente: https://dev.to/pixel-wraith/the-real-reason-your-prs-get-big-5cm3
Comunidad de aprendizaje opcional: https://t.me/GyaanSetuAi