pt.javascript.info
pt.javascript.info copied to clipboard
javascript-tutorial
Basic operators, math
Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review :ghost:
Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review :ghost:
@betofc89, obrigado pela contribuição ❤️.
Segue minha revisão 😀.
@jonnathan-ls , isto sim é uma revisão de altíssimo nível! Muito obrigado mesmo pelas observações, correções e sugestões. Sinto que vou melhorar o meu português com as suas ótimas observações! Vou fazer as mudanças sugeridas por você.
Eu confesso que sou novo no GitHub e decidi contribuir com esta tradução até mesmo como forma de eu entender melhor GitHub/Git. Eu sempre tive vontade de colaborar em algum projeto open source, e achei que este aqui seria um excelente começo.
Sobre a alteração de outro artigo fora de contexto
Eu reverti aqui Revert "Substituição por palavra mais precisa" esta sugestão que fiz.
Por eu ser inexperiente com GitHub é que, por exemplo, a sugestão de alteração que fiz para o 05-types acabou ficando aqui no pull request do 08-operators. Peço desculpas por isto, e digo de coração que não foi por desleixo/preguiça que isto acabou ocorrendo. Na verdade, ontem mesmo eu percebi isto, e cheguei à conclusão de que eu deveria ter aberto outro pull request para manter tudo mais organizado. Porém, como esta sugestão de alteração já havia caído aqui no PR do 08-operators e como eu não sabia como desfazer isto, eu preferi deixar assim, por medo de bagunçar alguma coisa. Eu não sabia que enquanto um pull request está aberto, qualquer outro commit criado cairia dentro deste PR (estou certo em concluir isto?).
Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review :ghost:
@jonnathan-ls , isto sim é uma revisão de altíssimo nível!
@betofc89 Obrigado pela apreciação, é um prazer receber feedback ❤️ Por favor, fique a vontade também para pontuar qualquer observação que eu tenha feito equivocadamente ><
Eu confesso que sou novo no GitHub e decidi contribuir com esta tradução até mesmo como forma de eu entender melhor GitHub/Git. Eu sempre tive vontade de colaborar em algum projeto open source, e achei que este aqui seria um excelente começo.
Muito bom, está certíssimo. 🔝
Não faz tanto tempo que passei a contribuir com a tradução deste repositório, pretendo rever conceitos em JavaScript e me aprofundar mais na linguagem. Como gostei bastante da plataforma, decidi contribuir traduzindo, e acabei ingressando como revisor também.
... Eu não sabia que enquanto um pull request está aberto, qualquer outro commit criado cairia dentro deste PR (estou certo em concluir isto?).
Pertinente a comentário e a pergunta ... está certo em sua conclusão, isto por que está trabalhando direto na master e qualquer alteração fará parte do fluxo de desenvolvimento que já está realizando.
Para gerenciar melhor o trabalho recomendo seguir algum paradigma de fluxo de desenvolvimento, comentei mais detalhadamente sobre isso aqui: https://github.com/javascript-tutorial/pt.javascript.info/pull/247#discussion_r946266677
@betofc89 Obrigado pela correção, revisão aprovada ...
Para aprovação de mesclagem, é necessário mais um revisor... com aprovação de um segundo avaliador a PR será aceita.
Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review :ghost:
Hi @betofc89 ,
I updated files in this branch according to the English version. As part of the pending tasks, could you also resolve conflicts?
Thanks, Osvaldo.
@odsantos since I don't have access to Beto's master branch, I forked his repo and fixed all remaining issues. Should I open a new PR?
@odsantos since I don't have access to Beto's
masterbranch, I forked his repo and fixed all remaining issues. Should I open a new PR?
Yes, please @Peruibeloko could you open it? Thanks.