What I Learned From Hacktoberfest
Hacktoberfest has really been good for me. This is not the first time I've contributed to open-source but this is the time I can say I truly understand open source and contributed much. My main target was learning to write test and I can say I accomplished that. I spent time with the Jest docs and I applied them to TheAlgorithms/Javascript repo, A repository for algorithms in JavaScript.
Add tests to Math #423
Welcome to JavaScript community
Describe your change:
- [x] Add tests
- [ ] Add an algorithm?
- [x] Fix a bug or typo in an existing algorithm?
- [ ] Documentation change?
Checklist:
- [x] I have read CONTRIBUTING.md.
- [x] This pull request is all my own work -- I have not plagiarized.
- [x] I know that pull requests will not be merged if they fail the automated tests.
- [ ] This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
- [x] All new JavaScript files are placed inside an existing directory.
- [x] All filenames should use the UpperCamelCase (PascalCase) style. There should be no spaces in filenames.
Example:
UserProfile.js
is allowed butuserprofile.js
,Userprofile.js
,user-Profile.js
,userProfile.js
are not - [ ] All new algorithms have a URL in its comments that points to Wikipedia or other similar explanation.
- [ ] If this pull request resolves one or more open issues then the commit message contains
Fixes: #{$ISSUE_NO}
.
Add Algorithms to Math with tests #429
Welcome to JavaScript community
Describe your change:
- [x] Add an algorithm?
- [ ] Fix a bug or typo in an existing algorithm?
- [ ] Documentation change?
Checklist:
- [x] I have read CONTRIBUTING.md.
- [x] This pull request is all my own work -- I have not plagiarized.
- [x] I know that pull requests will not be merged if they fail the automated tests.
- [ ] This PR only changes one algorithm file. To ease review, please open separate PRs for separate algorithms.
- [x] All new JavaScript files are placed inside an existing directory.
- [x] All filenames should use the UpperCamelCase (PascalCase) style. There should be no spaces in filenames.
Example:
UserProfile.js
is allowed butuserprofile.js
,Userprofile.js
,user-Profile.js
,userProfile.js
are not - [x] All new algorithms have a URL in its comments that points to Wikipedia or other similar explanation.
- [ ] If this pull request resolves one or more open issues then the commit message contains
Fixes: #{$ISSUE_NO}
.
Also, I made some typo corrections to the New Nodejs Learning guide, nodejs.dev. I was reading it and I contributed to it. The project is stable to me, but there are more features to be implemented and I want to help with that.
And finally added some utility functions to 30secondsofcode.org. Some where removed but not all and the pull was merged.
Add isNode, randomIpAddress and isRegularExpression #1526
Function to check if the environment is node Function to generate a random Ip Address. Function to check for Regular Expressions
Thank you Open Source, Thank you Hacktoberfest. I want to contribute to more open source.
Top comments (0)