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

[pt-BR] Translate about/index.md to pt-br #1974

Merged
6 commits merged into from
Jan 5, 2019

Conversation

Drieger
Copy link
Contributor

@Drieger Drieger commented Jan 2, 2019

Translated about/index.md to pt-br.

Refs: nodejs/nodejs-pt#71

/cc @nodejs/nodejs-pt

Copy link
Contributor

@glaucia86 glaucia86 left a comment

Choose a reason for hiding this comment

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

Pequenos ajustes. Muito boa a tradução. Parabéns @Drieger ! :)

[Blocking vs Non-Blocking]: https://nodejs.org/en/docs/guides/blocking-vs-non-blocking/
[`child_process.fork()`]: https://nodejs.org/api/child_process.html#child_process_child_process_fork_modulepath_args_options
[`cluster`]: https://nodejs.org/api/cluster.html
[event loop]: https://nodejs.org/en/docs/guides/event-loop-timers-and-nexttick/

Choose a reason for hiding this comment

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

Podemos substituir os links acima por versões em português (se os tivermos)?

Copy link
Contributor

Choose a reason for hiding this comment

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

Boa ideia @gustavohenke

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Concordo @gustavohenke, mas não encontrei a versão em português, acho que ainda não temos elas traduzidas. A medida que tivermos, podemos ir atualizando esse documento.

@gustavohenke
Copy link

I didn't mean to early approve this 🤦‍♂️
Please make sure @glaucia86's reviews are resolved before merging 😃

glaucia86 and others added 5 commits January 3, 2019 08:28
Co-Authored-By: Drieger <drbeckert@gmail.com>
Co-Authored-By: Drieger <drbeckert@gmail.com>
Co-Authored-By: Drieger <drbeckert@gmail.com>
Co-Authored-By: Drieger <drbeckert@gmail.com>
Co-Authored-By: Drieger <drbeckert@gmail.com>
@Drieger
Copy link
Contributor Author

Drieger commented Jan 3, 2019

@glaucia86 all suggestions were addressed. Could you take a look again?

@glaucia86
Copy link
Contributor

@glaucia86 all suggestions were addressed. Could you take a look again?

Let me see here again... a minute.

Copy link
Contributor

@glaucia86 glaucia86 left a comment

Choose a reason for hiding this comment

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

LGTM 👍
Excelent @Drieger

Copy link
Contributor

@cezaraugusto cezaraugusto left a comment

Choose a reason for hiding this comment

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

++

This pull request was closed.
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.

5 participants