Discusión de code review
Code review discussion
Llamada rápida para resolver comentarios en un pull request. Hay que defender decisiones, aceptar cambios y pedir ayuda concreta.
- Empieza siempre agradeciendo la review. Pone tono positivo.
- Defiende tus decisiones con datos, no con orgullo.
- 'Good catch' y 'fair point' son tus mejores amigas — úsalas mucho.
- Si no entiendes un comentario, pregunta en lugar de adivinar.
- Distingue nitpicks (estilo) de blockers (lógica). Atiende los blockers primero.
- Si el reviewer insiste, suele tener razón. No mueras en cada colina.
Thanks for the review. Let me explain my reasoning.
Gracias por la review. Te explico mi razonamiento.
Want to start with the comments at the top of the file?
¿Empezamos con los comentarios del principio del archivo?
I left a few replies, but happy to talk it through.
Dejé algunas respuestas, pero podemos hablarlo.
PR (pull request)
petición para integrar código
to merge
integrar en la rama principal
comment thread
hilo de comentarios
edge case
caso límite
to refactor
refactorizar
nitpick
comentario menor / detallito
to mock
simular una dependencia en tests
trade-off
compromiso / concesión
to push back
discrepar con educación
to address a comment
atender / resolver un comentario
blocking comment
comentario que bloquea el merge
LGTM (looks good to me)
me parece bien
“LGTM, ready to merge.”
Could we go to the line you mean?
¿Vamos a la línea de la que hablas?
Sorry, I lost track of which file we were in.
Perdón, me perdí en qué archivo estábamos.
Can you share your screen so I can see the comment?
¿Puedes compartir pantalla para que vea el comentario?
Let me re-read it for a second.
Déjame releerlo un segundo.
Sorry, could you repeat that, please?
Perdón, ¿puedes repetir eso, por favor?
Could you say that a bit more slowly?
¿Puedes decirlo un poco más despacio?
Could you clarify what you mean by that?
¿Puedes aclarar a qué te refieres?