2️⃣📖🕵️‍♂️ LECTURA: How to Make Your Code Reviewer Fall in Love with You

IslasGECI / seleccion_analista_2022_mvb

🐍 Mario Villasante Barahona

GNU Affero General Public License v3.0

0 stars 0 forks source link

2️⃣📖🕵️‍♂️ LECTURA: How to Make Your Code Reviewer Fall in Love with You #9

Closed devarops closed 2 years ago

devarops commented 2 years ago

¿Por favor podrías leer esta nota?

How to Make Your Code Reviewer Fall in Love with You

Te pido por favor que me digas:

¿Cuál parte te gustó más?
¿Cuál parte te gustó menos?
¿Cuál parte te resultó más clara?
¿En cuál parte tuviste más dudas?

mvillasante commented 2 years ago

Lo que más me gusta es el humor :smile_cat:

Creo que es un buen artículo porque toma en cuenta muchos aspectos sociales, que los artículo anteriores (más técnicos) dejan de lado.

Una cosa en la que siempre tengo dudas es:

Si ya solicité una re-review ¿es necesario que escribir mensajes en la conversación del PR?.

A veces pienso que esto puede interpretarse como falta de paciencia de mi parte. Supongo que debo estar atento a esto, para (como dice el artículo) "quien lleva la estafeta".

devarops commented 2 years ago

Si ya solicité una re-review ¿es necesario que escribir mensajes en la conversación del PR?.

Esa es una buena pregunta. La respuesta es sí, por lo general.

Pero en los primeros 3 PR de este examen no será necesario porque, en este caso, los PR están asociados a un issue particular que yo te pedí. Aquí yo tengo el contexto completo. Pero en PR posteriores (del 4 en adelante) yo no sabré qué intentas hacer; entonces sí será necesario explicar en la descripción del PR.