Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tradução Capítulo 10 até linha 363 #398

Merged
merged 10 commits into from
Nov 14, 2018
Merged

Tradução Capítulo 10 até linha 363 #398

merged 10 commits into from
Nov 14, 2018

Conversation

Morette
Copy link

@Morette Morette commented Jul 10, 2018

Seção de Módulos e Pacotes traduzida.
Até linha 151.

Morette added 3 commits July 10, 2018 09:20
tradução realizada até linha 75
das linhas 76 até 151
Copy link
Contributor

@joaostein joaostein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

valeu, @Morette. adicionei algumas notas de melhorias.

10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
10_modules.md Outdated Show resolved Hide resolved
@Morette
Copy link
Author

Morette commented Jul 11, 2018

Desculpa a pergunta, sou meio novo aqui no GitHub, mas como devo prosseguir agora?
Fecho esse pull request? Os seus comentários você já corrigiu no texto ou eu preciso corrigir e mandar outro pull?

@joaostein
Copy link
Contributor

joaostein commented Jul 11, 2018

@Morette os comentários que eu deixei são apenas para sua referência, eles não são adicionados no arquivo de texto final do livro.

Para adicionar as mudanças efetivamente, basta fazê-las da mesma forma que fez anteriormente, adicionando um (ou vários) commits e fazendo o push para o mesmo branch que você está trabalhando.

Quando estiver finalizado, é só comentar que, na sequência, eu vou dar mais uma revisada para ver se encontro mais detalhes que podemos melhorar e, se não encontrar mais nada, eu dou merge no seu pull request.

se tiver dúvida, só avisar.

@joaostein joaostein changed the title 3rd Tradução Capítulo 10 Jul 14, 2018
Copy link
Contributor

@joaostein joaostein left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Valeu, @Morette!

já dei merge. se for continuar traduzindo, é só ir enviando PRs.

10_modules.md Outdated Show resolved Hide resolved
@joaostein joaostein changed the title Tradução Capítulo 10 Tradução Capítulo 10 até linha 363 Nov 14, 2018
@joaostein joaostein merged commit fcbced5 into braziljs:3rd Nov 14, 2018
@joaostein
Copy link
Contributor

ref #409

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants