Sproken
B1Inglês·20 cartões

Code review em inglês sem climão

Comentar com tato, separar sugestão de bloqueio e receber crítica numa revisão de código.

A cultura americana de code review separa a pessoa do código: você comenta a linha, não o autor. O jargão é comum: nit = observação pequena e cosmética, blocking = objeção que impede o merge, LGTM = sinal verde. O tom costuma ser bem mais suavizado do que o brasileiro espera — « I might be wrong, but » abre muita crítica séria.

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

    Small nit, maybe rename this variable.

    Detalhezinho: talvez renomear essa variável.

  2. 2

    Just a suggestion, not blocking.

    Só uma sugestão, nada que bloqueie.

  3. 3

    I think this could be simplified a bit.

    Acho que isso poderia ser um pouco simplificado.

  4. 4

    What do you think about using a map here instead?

    O que você acha de usar um map aqui?

  5. 5

    Is there a reason for doing it this way?

    Tem algum motivo para fazer desse jeito?

  6. 6

    I might be wrong, but this looks duplicated.

    Posso estar enganado, mas isso parece duplicado.

  7. 7

    This part is a little hard to follow for me.

    Essa parte está meio difícil de acompanhar para mim.

  8. 8

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

    Eu gostaria de ver um teste para esse caso.

  9. 9

    This will break if the list comes back empty.

    Isso vai quebrar se a lista vier vazia.

  10. 10

    This one's blocking, I'm afraid.

    Esse aqui é bloqueante, infelizmente.

  11. 11

    Let's hash this out on a call.

    Vamos resolver isso numa call.

  12. 12

    Good catch, I didn't think of that.

    Bem observado, eu não tinha pensado nisso.

  13. 13

    You're right, I'll fix it.

    Você tem razão, vou corrigir.

  14. 14

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

    Prefiro deixar como está, e explico por quê.

  15. 15

    Let's open a ticket for that later.

    Vamos abrir um ticket para isso depois.

  16. 16

    That's out of scope for this PR.

    Isso está fora do escopo deste PR.

  17. 17

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

    Eu resolvo isso num PR de continuação.

  18. 18

    This looks good to me overall, nice work.

    No geral, para mim está bom, belo trabalho.

  19. 19

    Approved on my side, feel free to merge it.

    Aprovado da minha parte, pode fazer o merge.

  20. 20

    Thanks a lot for the thorough review on this.

    Muito obrigada pela revisão tão caprichada.