-
Notifications
You must be signed in to change notification settings - Fork 48
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
Variables #30
Variables #30
Conversation
Thank you for submitting the PR. I will review it by tomorrow. |
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.
Hey, @ashpratap007 added a few comments. Overall a good job. Thank you for contributing towards a good cause. Let's make the changes, then we will merge it. Let me know if I can help with anything.
1-js/02-first-steps/04-variables/2-declare-variables/solution.md
Outdated
Show resolved
Hide resolved
1-js/02-first-steps/04-variables/2-declare-variables/solution.md
Outdated
Show resolved
Hide resolved
Please make the requested changes. After it, add a comment "/done". |
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.
Thanks for the wonderful review. This being my first open-source contribution, kindly bear with me.
/done |
Hey, It was awesome! Merging this PR. Congrats for your first open-source contribution 🎉🎉. Hoping to see many more from you in the coming days. Thank you. |
Thank you 💖 I updated the Progress Issue #1 🎉 🎉 🎉 |
Hindi translations for topic "Variables".