DEV Community

sarabadu
sarabadu

Posted on

El Peer Review no sirve para nada (Spanish)

Bueno, en realidad no creo que no sirva para nada, de hecho, el peer review me parece una de las herramientas más productivas para la interacción entre compañeros, el crecimiento de un equipo y la calidad de nuestro trabajo.

Sin embargo, en muchos equipos de trabajo observe como menospreciaban esta práctica o bien la habían dejado completamente de lado. ¿Pero cuáles son los problemas?

No tenemos tiempo para eso

Bueno esto lo escuche muchas veces tanto para los peer review como para los tests unitarios.
Nunca vas a tener tiempo para las tareas que no les destines tiempo.

Cuando el problema es el tiempo necesitaremos decisiones fuertes y de largo plazo. Tanto el peer review como el testing unitario o el TDD, son prácticas que demuestran resultados cuando son constantes, se mantienen en el tiempo y se hacen costumbre.

¿Qué importancia tiene la calidad?
¿Qué tan valioso es la discusión entre colegas sobre el trabajo que estamos realizando?
¿Cuánto tiempo sería aceptable dedicarles a estas prácticas?

¿Y qué puedo hacer entonces?

  • Adapta el proceso (o crea un proceso) de desarrollo para que incluya el Peer Review y revisa que se cumpla.
  • ¿Necesitas convencer a alguien? Medí los tiempos de desarrollo y peer review
  • Pon metas a largo plazo

Y por último ten en cuenta que Arreglar cosas que no tuvieron peer review lleva tiempo también!!

No me gusta que me muestren mis errores

Siempre puede existir algo de incomodidad a la hora de marcar y que nos marquen errores, pero tenemos que poner el foco en el objetivo por el cual hacemos estas practicas

El peer review es para que nosotros, como equipo, tengamos mejores resultados

Ese es el objetivo, ¡si solo ese! En el peer review se pueden encontrar errores, también para compartir puntos de vista, validar si lo que hicimos se entiende y aprender del otro. Si nos enfocamos en solo buscar y encontrar errores, estamos desperdiciando la posibilidad de crecer en otros aspectos.

Por otro lado, acá juega mucho el cómo hacemos el peer review y como lo recibimos.

Cuando hagas un Peer Review:

  • Intenta entender que decisiones tomo tu colega
  • Toma nota de que cosas harías distinto y por que
  • Intenta buscar todos los detalles posibles: mala identación, nombre de variables, ¡¡todo sirve!!

Pero tu trabajo no termina ahí, si dejamos esto anotado y nos desentendemos estamos haciendo solo la mitad del trabajo...

  • Júntate con quien realizo el desarrollo y revisen tus notas juntos
  • Intenta reforzar "intentemos mejorar juntos con esto" para mantener el objetivo en mente
  • Mide tus palabras, ten especial cuidado en cómo te comunicas frases como "acá te equivocaste", "esto está mal" son fácilmente reemplazable con "yo esto lo haría diferente", "esto se ve raro, revisemos", "no entendí esta parte"

Pero si algo está mal, está mal y yo le voy a decir porque un error es un error y etc....

Si, si, pero ¿qué cuesta modificar un poco tus palabras para lograr el mismo objetivo? no hablo de decir que algo está bien cuando no lo es, ni que algo es aceptable cuando no lo es... hablo de mostrar eso mismo de la manera mas amable que se te ocurra.

Y cuando recibas un Peer Review

  • Tu trabajo no está terminado, solo es un boceto que está a punto de perfeccionarse. Si tomas como una revisión de algo que ya estaba "terminado" puede ser chocante. Mientras desarrollas piensa "aún faltan los cambios del peer"
  • Tu colega no es el enemigo, el enemigo es el llamado a las 3am por un bug, o el código spaghetti de hace 3 meses (que sí, vos escribiste)
  • Da tu opinión no tomes el peer como una lista de cambios y ya, discute con el peer porque tomaste una decisión, porque elegiste ese nombre, porque te equivocaste ahí.

El Peer Review puede impactar negativamente en mi carrera

EL PEER REVIEW NO PUEDE SER MÉTRICA PARA MEDIR PERFORMANCE. -necesito mayúsculas más grandes -

Bueno soy encargado de los Peer Reviews

Si tu puesto es Jefe, PM, CEO, o simplemente el que le gusta seguir métricas para ver como trabaja la gente, te recomiendo lo siguiente:

  • No tomes los errores de un peer review como indicador de performance de una persona, solo lograras que nadie quiera encontrar errores, puede que al principio veas algún dato, pero con el tiempo no vas a tener información.

  • Cuando planifiques prácticas de Peer Review, intenta plantearlas dentro del proceso de desarrollo, antes de "los últimos retoques" y antes de hacer las pruebas finales. si se planifican al final genera una sensación de retrabajo que no es necesaria

Pero encontré que Alejandra le encontró 500 errores a Anselmo, ¿Qué hago me quedo de brazos cruzados?

No, de ninguna manera, en ese caso felicita a Alejandra, está haciendo muy buen trabajo.
Anselmo solo mostró un borrador de su trabajo, es esperable que se encuentren errores.

Top comments (0)