Skip to content

Error handling, "try...catch" - #194

Merged
mahdyar merged 17 commits into
javascript-tutorial:masterfrom
mahdiHash:master
Feb 6, 2022
Merged

Error handling, "try...catch"#194
mahdyar merged 17 commits into
javascript-tutorial:masterfrom
mahdiHash:master

Conversation

@mahdiHash

@mahdiHashmahdiHash commented Feb 4, 2022

Copy link
Copy Markdown
Contributor

Please review and comment/commit if anything is wrong.

@javascript-translate-bot
javascript-translate-bot requested a review from a teamFebruary 4, 2022 18:22
@javascript-translate-botjavascript-translate-bot mentioned this pull request Feb 4, 2022
Open
mahdiHash
mahdiHash commented Feb 4, 2022
View reviewed changes
Comment thread1-js/10-error-handling/1-try-catch/article.md
mahdiHash
mahdiHash commented Feb 4, 2022
View reviewed changes
Comment thread1-js/10-error-handling/1-try-catch/article.md
mahdiHash
mahdiHash commented Feb 4, 2022
View reviewed changes
Comment thread1-js/10-error-handling/1-try-catch/article.md
mahdiHash
mahdiHash commented Feb 4, 2022
View reviewed changes
Comment thread1-js/10-error-handling/1-try-catch/article.md Outdated
mahdiHash
mahdiHash commented Feb 4, 2022
View reviewed changes
Comment thread1-js/10-error-handling/1-try-catch/article.md Outdated
mahdiHash
mahdiHash commented Feb 4, 2022
View reviewed changes
Comment thread1-js/10-error-handling/1-try-catch/article.md
mahdiHash
mahdiHash commented Feb 4, 2022
View reviewed changes
Comment thread1-js/10-error-handling/1-try-catch/article.md
@mahdiHash
mahdiHash marked this pull request as ready for review February 4, 2022 18:26
mahdyar
mahdyar requested changes Feb 5, 2022
View reviewed changes

@mahdyarmahdyar 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.

Please review the suggested change(s) and request another review.

@javascript-translate-bot

javascript-translate-bot commented Feb 5, 2022

Copy link
Copy Markdown

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

@mahdiHash@mahdyar
Apply suggestions from code review
1449c45
Co-authored-by: Mahdyar Hasanpour <thisismahdyar@gmail.com>
@mahdiHash

mahdiHash commented Feb 5, 2022

Copy link
Copy Markdown
ContributorAuthor

Thanks for your review, @mahdyar.
/done

@javascript-translate-bot
javascript-translate-bot requested a review from a teamFebruary 5, 2022 19:04
@mahdyar

mahdyar commented Feb 6, 2022

Copy link
Copy Markdown
Member

Thank you, @mahdiHash!

@mahdyar
mahdyar merged commit fd8848b into javascript-tutorial:masterFeb 6, 2022
@javascript-translate-bot

javascript-translate-bot commented Feb 6, 2022

Copy link
Copy Markdown

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

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

Reviewers

@mahdyarmahdyarmahdyar approved these changes
+1 more reviewer
@BeroozamBeroozamBeroozam left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

needs +1

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@mahdiHash@javascript-translate-bot@mahdyar@Beroozam