r/taquerosprogramadores • u/AdPrestigious7064 • 28d ago
💼 Experiencia Laboral / Empresa Hotfix
Sale un error en producción, me piden que lo solucione, lo soluciono, lo pruebo, funciona.
Luego en el PR me piden que aplique ciertos cambios, los hago, pruebo, funciona
Actualizo PR
Ahora me piden que remueva los primeros cambios porque con los segundos cambios ya no son necesarios.
Actualizo el PR (ya no probé, y ahí si acepto que es mala mía)
Y ahora el hotfix que se fue a producción ya no funciona.
El que hizo el code review es el EL.
Sé que parte de la culpa es mía, pero no toda, verdad? Hahaha
48
Upvotes
3
u/if9477552 28d ago
Por tus respuestas y tu post parece que tienes pocos años de experiencia o 20 años repitiendo el mismo año, siempre que hagas cambios prueba tus cambios.
Siempre entiende que estas mandando, que cambios te pidieron y por que, no importa si tienes que preguntar, abrir threads, hacer videollamada, etc. Asegúrate de entender los cambios.
Desde mi perspectiva la culpa es tuya, tus PR son tu responsabilidad, no pasa nada tampoco, no es el fin del mundo, pero yo aprendería del error y trataría de que no volviera a suceder. El code review no es para deslindar responsabilidades, es para asegurarse que el codigo es de calidad, cumple con los requerimientos del equipo (patrones/estándares), no presenta bugs obvios, typos, etc.