owc blog 2
hat Code Review Changed Me During my senior year of college, I decided to contribute to an open source project. I picked a small “problem” and submitted my solution as a pull request (PR) on GitHub. My code worked, passed the tests, and honestly, I thought I did a good job. But the first comment that came gave me pause: “Your function is great, but it’s a bit too much work. Can you consider reducing it to a single responsibility?” This comment seemed simple, but I realized at that moment: It’s not enough for code to just work. It also needs to be readable, maintainable, and understandable to others. Later comments pointed out that variable names were vague, I was skipping some test cases, and I lacked modularity. The comments were not destructive, but they were clear. The criticisms were open-ended questions: “Why was this part done this way?” “Have you considered this approach as an alternative?” I was initially discouraged, but when I went back and reviewed my code again, I agre...