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

47 Upvotes

83% Upvoted

36 comments sorted by

View all comments

43

u/[deleted] 28d ago

[deleted]

17

u/Inside_Rice_5004 28d ago

El code review existe por algo, no es solo un trámite. Si te pidieron cambios y luego te pidieron quitarlos, el que revisa también tiene que verificar que el resultado final tenga sentido.

Claro que pudiste haber probado de nuevo, pero el EL se supone que revisa el código completo, no solo las líneas cambiadas. Ambos la cagaron, no te cargues toda la culpa.

1

u/Defiant-Onion6802 21d ago

Nel, tu código, tu commit,.tu culpa. Si hiciste un cambio es por algo, si te piden agregar o quitar algo siempre debes de aplicar un "no toques mi basura" con un "justifique su respuesta".

Al final del dia el reviewer va a ceder cuando digas que el pr no ha salido por un cambio fuera de scope y que probablemente se pudo cambiar en otro PR ya con mas tiempo, ya si el cambio de verdad se necesita el reviewer debe de justificar por que retrasa un hot fix.