Sproken
B1Inglés·20 tarjetas

Code review en inglés

Escribir un comentario con tacto, distinguir sugerencia de bloqueo y aceptar la crítica.

La cultura estadounidense de code review separa a la persona del código: se comenta la línea, no al autor. La jerga es común: nit = detalle cosmético menor, blocking = objeción que impide el merge, LGTM = luz verde. Ojo: « review » se pronuncia « ri-VIU » y el sustantivo « PR » se dice « pi-AR ».

Two colleagues enjoying a casual conversation during a coffee break in a modern office setting.
Foto: Rene Terp en Pexels
  1. 1

    Small nit, maybe rename this variable.

    Un detallito: tal vez renombrar esta variable.

  2. 2

    Just a suggestion, not blocking.

    Solo es una sugerencia, no es bloqueante.

  3. 3

    I think this could be simplified a bit.

    Creo que esto se podría simplificar un poco.

  4. 4

    What do you think about using a map here instead?

    ¿Qué te parece usar un map aquí en su lugar?

  5. 5

    Is there a reason for doing it this way?

    ¿Hay alguna razón para hacerlo así?

  6. 6

    I might be wrong, but this looks duplicated.

    Puede que me equivoque, pero esto parece duplicado.

  7. 7

    This part is a little hard to follow for me.

    Esta parte me cuesta un poco seguirla.

  8. 8

    I'd like to see a test for this case.

    Me gustaría ver un test para este caso.

  9. 9

    This will break if the list comes back empty.

    Esto va a fallar si la lista viene vacía.

  10. 10

    This one's blocking, I'm afraid.

    Este sí es bloqueante, lo siento.

  11. 11

    Let's hash this out on a call.

    Resolvamos esto en una llamada.

  12. 12

    Good catch, I didn't think of that.

    Bien visto, no se me había ocurrido.

  13. 13

    You're right, I'll fix it.

    Tienes razón, lo corrijo.

  14. 14

    I'd rather keep it as is, and here's why.

    Preferiría dejarlo como está, y te explico por qué.

  15. 15

    Let's open a ticket for that later.

    Abramos un ticket para eso más adelante.

  16. 16

    That's out of scope for this PR.

    Eso está fuera del alcance de este PR.

  17. 17

    I'll handle it in a follow-up PR.

    Me encargo de eso en un PR aparte.

  18. 18

    This looks good to me overall, nice work.

    En general me parece bien, buen trabajo.

  19. 19

    Approved on my side, feel free to merge it.

    Aprobado de mi lado, puedes hacer el merge cuando quieras.

  20. 20

    Thanks a lot for the thorough review on this.

    Muchas gracias por la review tan completa.