Hacker News new | ask | show | jobs
by codewiz 1681 days ago
Well, what would happen if everyone prioritized writing code over reviewing their coworkers' code?

Since junior engineers and new hires are not yet able to review code effectively, senior engineers end up reviewing a lot more code than they write.

I think recognizing and rewarding this behavior in performance reviews is entirely reasonable for any tech company that wants a sustainable codebase.

1 comments

The problem isn't that code reviews get recognized and rewarded, it's that once a change is merged you can't review it, so people "race" to review changes before their colleagues get to it. That's not a healthy system. It's better to let people manage their time to promote flow, letting them batch up code reviews at a certain time of day if that's more effective.

I'll just add that nowhere I've worked has counted code reviews in this way and they still get done, so it's not a disaster when you don't officially penalize people for their review count.