Skip to content

Conversation

@mahdiHash
Copy link
Contributor

Please review and comment/commit if anything is wrong.

Line 7 needs more attention
Line 19 needs more attention.
Line 115, 127 and 139 need more attention.
Line 168 needs more attention.
Line 206 needs more attention.
@mahdiHash mahdiHash marked this pull request as ready for review January 8, 2022 13:36
Copy link
Member

@mahdyar mahdyar left a comment

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 for another review.

Copy link
Member

@mahdyar mahdyar left a comment

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 for another review.

@javascript-translate-bot

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

Co-authored by: Mahdyar Hasanpour <[email protected]>
Co-authored by: Mahdyar Hasanpour <[email protected]>
Co-authored by: @Beroozam
@mahdiHash
Copy link
Contributor Author

mahdiHash commented Jan 10, 2022

Thanks for your reviews @mahdyar @Beroozam
/done

Copy link
Member

@mahdyar mahdyar left a comment

Choose a reason for hiding this comment

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

LGTM!

@mahdyar mahdyar merged commit c6eeb0a into javascript-tutorial:master Jan 10, 2022
@javascript-translate-bot

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants