Skip to content

docs(pt-br): translate installation page #666

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 3 commits into from
Closed

docs(pt-br): translate installation page #666

wants to merge 3 commits into from

Conversation

ErickRock
Copy link

Issue #555

docs(pt-br): Translate installation page

Tradução da página de Instalação

jotaRenan
jotaRenan reviewed Apr 26, 2023
View reviewed changes
Copy link
Contributor

@jotaRenan jotaRenan 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 sua tradução, Erick! Ressaltei dois pormenores que chamaram a atenção

@jotaRenan
Copy link
Contributor

PS: vc pra sempre terá a pr numero 666 desse repo 😈

@ErickRock
Copy link
Author

ErickRock commented Apr 26, 2023 •
edited
Loading

PS: vc pra sempre terá a pr numero 666 desse repo 😈

kkkk logo nas minhas contribuições iniciais, espero que não seja nenhum sinal.
Commitei as mudanças, atualizando.

cauli
cauli approved these changes Apr 26, 2023
View reviewed changes
Copy link
Contributor

@cauli cauli left a comment

Choose a reason for hiding this comment

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

LGTM

@NivaldoFarias
Copy link
Contributor

dupe PR #661

Como ainda não foi designado os autores para cada página, provavelmente não tem problema existir mais de um PR alterando o mesmo arquivo. Assim que alguém for designado a gente revisa comparando as traduções pra deixar o texto final redondinho.

ErickRock
ErickRock commented Apr 26, 2023
View reviewed changes
Copy link
Author

@ErickRock ErickRock left a comment

Choose a reason for hiding this comment

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

Sugestões aplicadas

@jhonmike jhonmike added the duplicate This issue or pull request already exists label Apr 27, 2023
@jhonmike
Copy link
Member

@ErickRock sinto muito, eu tento atualizar a issue de tradução o mais rápido possível, mas nem sempre da tempo, acabou gerando um PR duplicado aqui, espero que vc não fique chateado, mas vou fechar esse PR, e fico torcendo que vc continue traduzindo outras páginas!!

muito obrigado pelo esforço!!

@jhonmike jhonmike closed this Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jotaRenan jotaRenan jotaRenan left review comments

@cauli cauli cauli approved these changes

Assignees
No one assigned
Labels
duplicate This issue or pull request already exists
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@ErickRock @jotaRenan @NivaldoFarias @jhonmike @cauli