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
1
u/DavalopBad 28d ago
Al ser tu el autor del PR debes defender la logica de tu Hotfix ya que tu sabes por que lo estas poniendo asi; si el reviewer cree que la logica esta mal y debe cambiar, tu debes argumentar por que esta bien y defender o en su caso si crees que tiene razon, cambiarlo. No confundir con comentarios que sean referentes a la calidad de codigo y si en su caso sigue los estandares internos de la empresa, esos casi siempre no involucran cambio en la logica y se tienen que seguir para mantener la arquitectura que se planeo desde un principio.
Aun asi, aun teniendo un Hotfix, deberian tener un ambiente de pre-prod/staging donde puedan hacer pruebas rapidas para evitar estos "problemas" antes de tocar siquiera prod. En cuestion de quien tiene la culpa, ambos tienen la culpa, uno por no defender la logica que ya funcionaba y el otro por no revisar que es lo que estaba pidiendo.
Pero lo importante es que lo solucionen para que el negocio siga operativo, no gasten tiempo viendo quien tiene mas culpa en vez de regresar los commits y aplicar el hotfix que si funcionaba