Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

1. I stopped seeing my work as a labor of love a long time ago. I love what I do, but my passion is with the technology, not the day-to-day commits. My work is generally good and I'm confident that I'll continue to be valued, but my job output isn't some kind of masterwork (nor does anyone want it to be).

2. The whole point of code review is to find issues. If my code was perfect, we'd skip review. But it's not, and I accept that my mortal flaws will inevitably show in what I put up for review.

3. Criticisms are one of the few feedback mechanisms for your real, tangible job output. It's important to remember that you grow technically by others pointing out mistakes. If you got nothing but praise, you're not actively refining your skills. Catching mistakes before you submit for review is indeed a skill that must be honed.

If the feedback you get is snarky or mean spirited, that's a culture issue. Reply politely, thank them for pointing it out, ask if they have suggestions for what to do better if the right answer isn't obvious. Establish a norm of treating code reviews as a place for being decent. If you don't get respect in return, take it up with a manager: code reviews at work are no place for someone to make you feel belittled.



Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: