Sproken
B1Anglais·20 cartes

La code review en anglais

Formuler un commentaire avec tact, distinguer suggestion et blocage, accepter la critique.

La culture américaine de la review sépare la personne du code : on commente la ligne, pas l'auteur. Le jargon est courant : nit = petite remarque cosmétique, blocking = objection qui empêche le merge, LGTM = feu vert.

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

    Small nit, maybe rename this variable.

    Petit détail : peut-être renommer cette variable.

  2. 2

    Just a suggestion, not blocking.

    Juste une suggestion, rien de bloquant.

  3. 3

    I think this could be simplified a bit.

    Je pense que ça pourrait être un peu simplifié.

  4. 4

    What do you think about using a map here instead?

    Qu'est-ce que tu dirais d'utiliser une map ici ?

  5. 5

    Is there a reason for doing it this way?

    Il y a une raison de faire comme ça ?

  6. 6

    I might be wrong, but this looks duplicated.

    Je me trompe peut-être, mais ça a l'air dupliqué.

  7. 7

    This part is a little hard to follow for me.

    Cette partie est un peu difficile à suivre pour moi.

  8. 8

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

    J'aimerais voir un test pour ce cas.

  9. 9

    This will break if the list comes back empty.

    Ça va planter si la liste revient vide.

  10. 10

    This one's blocking, I'm afraid.

    Celui-là est bloquant, désolé.

  11. 11

    Let's hash this out on a call.

    Tranchons ça lors d'un appel.

  12. 12

    Good catch, I didn't think of that.

    Bien vu, je n'y avais pas pensé.

  13. 13

    You're right, I'll fix it.

    Tu as raison, je corrige.

  14. 14

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

    Je préférerais le laisser tel quel, et voici pourquoi.

  15. 15

    Let's open a ticket for that later.

    On ouvre un ticket pour ça plus tard.

  16. 16

    That's out of scope for this PR.

    C'est hors du périmètre de cette PR.

  17. 17

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

    Je m'en occupe dans une PR de suivi.

  18. 18

    This looks good to me overall, nice work.

    Dans l'ensemble, ça me paraît bien, beau boulot.

  19. 19

    Approved on my side, feel free to merge it.

    Validé de mon côté, tu peux la merger.

  20. 20

    Thanks a lot for the thorough review on this.

    Merci beaucoup pour cette review très complète.