-
-
Notifications
You must be signed in to change notification settings - Fork 113
Comparisons #245
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
Comparisons #245
Conversation
8f95a2e to
ddd7bd8
Compare
Remove English phrases. Probably forgotten.
@odsantos
odsantos
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi,
You did not translate "task.md", and in "solution.md", "apple" > "pineapple" might be also translated.
Please, fell free to tell where you disagree.
Thanks, Osvaldo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Eu uso uma extensão ("LTeX") que me auxilia na sintaxe e concordância do texto.
A expressão vir a ter foi identificada como uma perífrase, podendo ser simplificada.
Não há necessidade do vir a, vide exemplificação do motivo:
https://community.languagetool.org/rule/show/VIR_A_VERBO_VERBO?lang=pt-BR
Substitui o trecho por outra expressão que deixa mais nítido, tudo bem?
- ... Se uma variável puder a ter esses valores ... + ... Se uma variável tiver `null` ou `undefined` ...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jonnathan-ls eu recomendo usar a expressão Se uma variável for null ou undefined.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jonnathan-ls could you update your pull request to resolve this commit suggestion?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done @nazarepiedady 😄 (a6fd6a8)
javascript-translate-bot
commented
Dec 7, 2022
Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻
fc88e1f to
60d70a2
Compare
jonnathan-ls
commented
Oct 15, 2023
/done
Olá @odsantos, apliquei os ajustes, poderia avaliar por gentileza?
You did not translate "task.md", and in "solution.md",
"apple" > "pineapple"might be also translated.
Nessa parte, ao contemplar a tradução acaba invertendo a ordem das letras no dicionário
Então, foi preciso registrar o resultado como true
"maça" > "abacaxi" → true
- Comparação de dicionário, portanto, verdadeiro.
"m"é a maior que"a".
60d70a2 to
87320ca
Compare
odsantos
commented
Nov 18, 2023
@jonnathan-ls hope to do it as soon as possible. Thanks.
jonnathan-ls
commented
Dec 16, 2023
/done
Adicionei um novo commit (a6fd6a8) @odsantos e um comentário em uma de suas revisões: #245 (comment)
nazarepiedady
commented
Dec 20, 2023
Eu aplicarei as modificações, quanto as pequenos pontos que carecem de ajustes serão revisados e atualizados posteriormente, precisamos fazer este projeto avançar e não bloquear o mesmo por causa de alguns pequenos detalhes, é melhor aproveitar cada contribuição ainda que pequena que recebemos.
javascript-translate-bot
commented
Dec 20, 2023
Error: the article already translated in the Progress Issue #1
Comparisons
📝 Goal
☑ What was done?
Translate again, the article
🤔 Reason?
The article had already been translated according to PR number 15, and even if combined, the translation does not appear in the main branch.
I couldn't find the reason for the translation not being in the main branch, but I deduce that it may have been from some sync PR with the main project (EN), as the folder order number is different from when the PR was approved