-
-
Notifications
You must be signed in to change notification settings - Fork 35.8k
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
fix(curriculum): update solution to match seed #38828
Conversation
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.
LGTM 👍
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.
LGTM! 👍
@vsasvipul0605 Congrats on your first pull request (PR)! 🎉 Thank you for your contribution to the page! 👍 |
By the way, @vsasvipul0605, if you want to keep using your fork you'll need to get a new version of master from upstream, since it will be out of sync now this PR has been merged. |
@ojeytonwilliams okay and thanks for merging the pull request. This will motivate me to contribute more and more. |
and can anyone please tell me how can I become contributor in freecodecamp? |
Well, you're already a contributor now that this PR has been merged. As for anything beyond that, it's not really something I can talk about with any authority. Personally I just contributed PRs and helped out a bit in Gitter, but I don't think there's a formal process. |
…ish.md
Checklist:
Update index.md
)master
branch of freeCodeCamp.Closes #38829
According to recent changes on site, the arrow function is not there. So, here's the updated solution for the updated challenge.