-
Notifications
You must be signed in to change notification settings - Fork 100
New translation: upgrading-node-js.md #187
New translation: upgrading-node-js.md #187
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.
🎉 Obrigado pela tradução @thiagorowof!
Estou aprovando mas vou deixar uma sugestão aqui:
- Acho que ficaria melhor se atualizasse o exemplo para a versão mais atual do node, que no caso é a 12.
Mas não sei se precisa mesmo, o que vocês acham, @jessescn e @iaurg ?
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.
Muito boa a tradução @thiagorowof obrigado por contribuir. 🎉
É uma ideia boa, mas creio que nesse caso como é apenas um exemplo fica subentendido que deve-se utilizar a versão mais atual do Node no momento. E seguimos com a base do original. |
O pior que eu fiz isso na época na versão mais atualizada kkkkk Valeu de qualquer forma <3 |
Qual o objetivo dessa pull request?
Qual arquivo foi traduzido/corrigido?
https://github.com/gatsbyjs/gatsby-pt-BR/blob/master/docs/docs/upgrading-node-js.md
https://github.com/thiagorowof/gatsby-pt-BR/blob/traducao-upgrading-node-js/docs/docs/upgrading-node-js.md
Algum comentário em relação a tradução?