Skip to content

Debugging in the browser#399

Merged
nazarepiedady merged 8 commits into
javascript-tutorial:masterfrom
zenobiopereira:master
Jun 2, 2024
Merged

Debugging in the browser#399
nazarepiedady merged 8 commits into
javascript-tutorial:masterfrom
zenobiopereira:master

Conversation

@zenobiopereira

Copy link
Copy Markdown
Contributor

Translated it the best I could, please let me know if there are some kind of convention that I've broken or some kind of phrasing that doesn't seems good.

There's also the hello.js file that I assume it should not have function names translated so I didn't touch it.

@CLAassistant

CLAassistant commented Mar 17, 2023

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
2 out of 3 committers have signed the CLA.

✅ zenobiopereira
✅ odsantos
❌ Zenóbio


Zenóbio seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@odsantos

Copy link
Copy Markdown
Contributor

@zenobiopereira , it seems you took a translated article and are proposing changes.
Do you want your suggestions to be reviewed?
Thanks.

@zenobiopereira

Copy link
Copy Markdown
Contributor Author

Hey @odsantos , yeah, it seems like the topic already had been translated but at least for me, the text was using multiple terms in which seems like PT-PT, even though, I would be glad if you could review it as proposed changes and if that's not possible then I can jump to translate a different topic.

@odsantos

Copy link
Copy Markdown
Contributor

Hi @zenobiopereira , yes I'll try to review it as soon as possible. Thanks.

@odsantos odsantos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hi @zenobiopereira ,

Here is a review.
Please don't hesitate to tell me where you disagree.

Thank you,
Osvaldo.

Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated
Comment thread 1-js/03-code-quality/01-debugging-chrome/article.md Outdated

@odsantos odsantos left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hi @zenobiopereira ,

Here is a review.
Please don't hesitate to tell me where you disagree.

Thank you,
Osvaldo.

@javascript-translate-bot

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@zenobiopereira

zenobiopereira commented Mar 26, 2023

Copy link
Copy Markdown
Contributor Author

Hey @odsantos thank you for the review, I did applied most of your corrections since pretty much all of them added context and small needed corrections.

I've also left small comments in two requested changes just because I think the text as it is, already delivers the same meaning, please let me know what do you think in relation to those.

Thank you in advance.

@zenobiopereira

Copy link
Copy Markdown
Contributor Author

/done

@odsantos

Copy link
Copy Markdown
Contributor

Hi @zenobiopereira left a comment on the review point on discussion. Thanks.

@zenobiopereira

Copy link
Copy Markdown
Contributor Author

@odsantos
Applied all the changes, let me know if there's something else that might need to be changed.

Thank you in advance.

/done

@zenobiopereira zenobiopereira requested a review from odsantos April 12, 2023 18:11
@odsantos

Copy link
Copy Markdown
Contributor

Hi @zenobiopereira , will take a look as soon as possible. Thanks.

@jonnathan-ls jonnathan-ls left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Embora, ainda considero que o artigo está em um linguajar muito formal, entendo que a necessidade deveria vir de outro momento para refletirmos em relação as colocações de escrita que este repositório em PT pode vir a adotar.

Sendo assim, considero a solicitação: LGTM ( Looks good to me )

@jonnathan-ls jonnathan-ls self-requested a review June 4, 2023 22:29

@jonnathan-ls jonnathan-ls left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM (Looks Great To Me)

@javascript-translate-bot

Copy link
Copy Markdown

Please make the requested changes. After it, add a comment "/done".
Then I'll ask for a new review 👻

@nazarepiedady

Copy link
Copy Markdown
Member

@jonnathan-ls, the language should be formal because programming is science.

@nazarepiedady nazarepiedady left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It also looks good to me to be approved.

@nazarepiedady nazarepiedady merged commit 6cd1cce into javascript-tutorial:master Jun 2, 2024
@javascript-translate-bot

Copy link
Copy Markdown

Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants