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

New translation: upgrading-node-js.md #187

Merged
merged 2 commits into from
Apr 15, 2020

Conversation

thiagorowof
Copy link
Contributor

@thiagorowof thiagorowof commented Jan 30, 2020

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?

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?

@jay-jlm jay-jlm changed the title upgrading-node-js translate New translation: upgrading-node-js translate Apr 8, 2020
@jay-jlm jay-jlm changed the title New translation: upgrading-node-js translate New translation: upgrading-node-js.md Apr 8, 2020
@iaurg iaurg added the /docs File refering to /docs directory label Apr 13, 2020
Copy link
Contributor

@henriquepw henriquepw left a 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 ?

Copy link
Contributor

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

@iaurg
Copy link
Contributor

iaurg commented Apr 15, 2020

tada 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 ?

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

@iaurg iaurg merged commit c20dbe1 into gatsbyjs:master Apr 15, 2020
@thiagorowof
Copy link
Contributor Author

thiagorowof commented Apr 16, 2020

tada 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 ?

É 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
Mas ai (não estou reclamando, só comentando) com a demora da avaliação e tal aqui do PR, teve atualização na documentação, e mudança de versão na documentação....tanto que antes era node 6 se não me engano.

Valeu de qualquer forma <3

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.

4 participants