Skip to content
This repository was archived by the owner on Jan 20, 2025. It is now read-only.

New translation: glossary/node.md #222

Merged
merged 2 commits into from
Apr 13, 2020
Merged

Conversation

sosolidkk
Copy link
Contributor

Qual o objetivo dessa pull request?

  • Adição de uma nova tradução
  • Correção em uma tradução existente

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.

@jessescn jessescn added the /docs File refering to /docs directory label Mar 24, 2020
jessescn
jessescn previously approved these changes Mar 24, 2020
Copy link
Contributor

@jessescn jessescn left a 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.

@jay-jlm jay-jlm changed the title traslation of glossary/node.md New translation: glossary/node.md Apr 8, 2020
Copy link
Contributor

@jay-jlm jay-jlm left a 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.

@sosolidkk
Copy link
Contributor Author

@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.

@jessescn jessescn merged commit f3aeff8 into gatsbyjs:master Apr 13, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
/docs File refering to /docs directory
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants