Skip to content

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 😃

@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