-
Notifications
You must be signed in to change notification settings - Fork 100
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Por mim parece ótimo, aprovado.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jessescn @sosolidkk Existe um padrão para utilização ou nao de artigo na frente dos nomes de ferramentas como Gatsby e npm? Notei que nessa tradução não está sendo usado. Particularmente acho que a legibilidade fica prejudicada quando nao tem o artigo, mas nao incluí na revisão.
Não acho que exista um padrão, mas em algumas PRs em que eu fiz (no primeiro comentário de revisão ele remove o "do Gatsby"), as vezes observo o Rafael retirando o artigo antes de Gatsby ou nome próprios de frameworks e/ou ferramentas, então terminei não colocando mais. Mas posso evitar e voltar a colocar. |
Qual o objetivo dessa pull request?
Qual arquivo foi traduzido/corrigido?
node.md
Algum comentário em relação a tradução?
Traduzi
runtime
para interpretador, mas não tenho certeza se a tradução mais adequada.