I made my first open source contribution today, which is pretty exciting (for me). On Sunday night, I was working my way through Underscore‘s unit tests, which I’m porting to Bound.js to use for testing Underscore compatibility. I noticed that one of the unit tests for lastIndexOf() was actually calling indexOf(), most likely because someone had just copied and pasted the unit tests for indexOf() and then forgotten to change one method call.
Easy fix, so I forked Underscore, cut a branch and submitted a pull request. This morning Brad Dunbar approved and merged it, and now I’m an Underscore contributor.