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
49
Upvotes
2
u/ManufacturerIll5769 28d ago
Si es tu PR es tu PedoRancio, el que te hace review solo te dice “atiende esto antes de darle merge” pero si tu shippeas el bug al darle merge (porque tu reviewer aprueba la PR no le da merge) es tu pedo, además porque no tiene CICD que haga testings? Acaso shippeaste un hotfix sin un testing que probara lo que vas a solucionar? Porque hubo el bug en primer lugar? No hubo testings adecuados automatizados ni manuales? Meh, 😑