Inglês para programadores
O inglês que se usa para trabalhar em tecnologia: da entrevista à daily de todo dia, do code review ao Slack. Cada item traz tradução, exemplo e uma nota sobre quando usar. Toque no alto-falante para ouvir a pronúncia.
Crie uma conta para treinar com repetição espaçada e acompanhar o que já domina.
nitpick
implicância
This is just a nitpick, but `data` could be called `pendingOrders` to make the intent clearer.
Isso é só uma implicância, mas `data` poderia se chamar `pendingOrders` para deixar a intenção mais clara.
Prefixar um comentário com isso avisa que a mudança é opcional e não bloqueia a aprovação. Muito usado em revisão de código.
LGTM
está bom para mim, pode seguir
LGTM, feel free to merge once CI is green.
Está bom para mim, pode integrar assim que a esteira ficar verde.
Sigla de 'looks good to me'. Use só quando você realmente leu o código: aprovar no automático destrói a confiança do time em você como revisor.
suggestion
trecho de código proposto no comentário que o autor aplica com um clique
I left a suggestion with the exact wording, just click apply if you agree.
Deixei um trecho pronto com o texto exato, é só aplicar se você concordar.
Na revisão de código a palavra virou nome de recurso do GitHub, não é só ideia solta. Use para correção mecânica e guarde a discussão para o que importa.
typo
erro de digitação
Small typo in the comment above, it says 'recieve'.
Pequeno erro de digitação no comentário acima, está escrito 'recieve'.
Para erros triviais, ser direto é seguro e até preferido. Rodeio em erro de digitação só faz o comentário ficar longo à toa.
out of scope
fora do escopo
Switching this from callbacks to async/await feels out of scope for a bug-fix PR.
Trocar isso de callbacks para async/await parece fora do escopo para uma correção de bug.
Forma padrão de recusar trabalho extra numa revisão sem ofender: o problema não é a ideia, é o lugar dela.
blocking comment
observação que precisa ser resolvida antes de integrar o código
Only one blocking comment from me, everything else is optional.
Só um comentário meu impede a integração, todo o resto é opcional.
Marque explicitamente, senão o autor decide sozinho o que ignorar. O par é non-blocking, e alguns times prefixam com issue: ou nit: para deixar claro.
rubber-stamp
aprovar no automático, sem ler de verdade
Please don't rubber-stamp this one, the debounce logic on the search input is subtle.
Por favor, não aprove essa no automático, a lógica do debounce no campo de busca é sutil.
Expressão com carga negativa: descreve quem aprova sem ler. Nunca use para acusar um colega em público, no máximo para falar do processo do time.
drive-by review
comentário rápido de quem não foi designado e não vai acompanhar
Sorry for the drive-by review, I noticed this while browsing the repo.
Desculpa o comentário de passagem, eu vi isso enquanto olhava o repositório.
Pode ajudar ou irritar: sem contexto, vira palpite. Se opinar em código que não é seu, diga que não bloqueia e não espere que o autor negocie com você.
Could we extract this into a separate function?
Poderíamos extrair isso para uma função separada?
O imperativo puro ('Extract this into a function') soa como ordem em time internacional. Acrescentar 'please' no fim não resolve: o pedido continua sendo ordem e o 'please' ainda soa impaciente.
What do you think about moving this check to the service layer?
O que você acha de mover essa verificação para a camada de serviço?
Abrir espaço para a opinião do autor é o padrão em revisão de código em inglês. Você continua claro sobre o que quer, mas convida a discussão em vez de fechá-la.
This is a nit, feel free to ignore it.
Isso é bobagem minha, pode ignorar se quiser.
'Nit' é a forma curta e informal de implicância. A segunda metade da frase é o que importa: sem ela, o autor não sabe se pode mesmo ignorar.
Nice catch on the null check here.
Boa sacada nessa verificação de valor nulo.
Elogiar em revisão não é bajulação. Em time distribuído, comentário só negativo passa impressão de hostilidade, e um elogio específico por revisão já muda o tom.
Not blocking, but we might want to add a test for this branch.
Não impede a aprovação, mas talvez valha um teste para esse ramo do código.
Diga sempre se o comentário impede ou não a aprovação. Essa marcação evita que uma mudança fique parada por causa de um detalhe opcional.
I'd suggest using a map here to avoid the nested loop.
Eu sugeriria usar um mapa (estrutura chave-valor) aqui para evitar o laço aninhado.
'I'd suggest' é o meio-termo entre a ordem e o rodeio: registra a recomendação sem virar exigência, e o motivo vem sempre junto. Em JavaScript, diga 'a Map' ou 'a plain object' — 'map' sozinho confunde com o método array.map().
Can you help me understand why we need this ENABLE_LEGACY_CHECKOUT flag?
Você pode me ajudar a entender por que precisamos dessa chave ENABLE_LEGACY_CHECKOUT?
Bem melhor que 'Why did you do this?', que por escrito soa como cobrança. Se a resposta for boa, você aprendeu. Se não for, o próprio autor percebe.
I ran this locally and everything worked as expected.
Rodei isso na minha máquina e tudo funcionou como esperado.
Contar o que você testou dá peso à sua aprovação e mostra que a revisão não foi só leitura rápida das diferenças.
Approving so you are not blocked, but please address the comment before merging.
Aprovando para você não ficar travado, mas resolva o comentário antes de integrar.
Aprovação condicional: comum em times que confiam uns nos outros e não querem perder um dia de fuso. Só funciona se o time realmente cumprir a segunda parte.
Could you add a short comment explaining this regex?
Você poderia colocar um comentário curto explicando essa expressão regular?
Pedido concreto e pequeno. Revisão que pede 'melhore isso' sem dizer o quê gera retrabalho e irritação.
18 itens neste tópico.
Cada guia explica a situação inteira — o roteiro, o que se espera de você e os erros típicos de quem fala português.