News for hackers. Stuff that matters.
Re:I ruin developers’ lives with my code reviews and I'm sorry by marcinzm on Monday February 18, 2019 @02:14PM (#19193356)
I'd argue that if you have a toxic team culture then no matter what you do things will escalate. Code reviews are a focal point but if they didn't exist then the focal point would move somewhere else. For example, engineer A commits some code and then engineer B rewrites it and then they get into an argument about it during a meeting. Toxicity in code reviews is at least auditable so a pro-active manager can nip it versus someone going off in a meeting they weren't part of (in which case it becomes a she said/he said situation).
Re:I ruin developers’ lives with my code reviews and I'm sorry by matfil on Tuesday February 19, 2019 @04:51AM (#19197837)
I don't think "toxic team culture" is the only way code reviews get frustrating. Other issues include frustrating waiting for reviews to be completed or the flip-side of people being knocked out of flow to do a review because "reviews are top priority". Relatedly, arguments about what the appropriate size is for a reviewable change.

The one alternative that (some) code-review advocates seem willing to accept is XP-style full-time pair programming. But that leaves even less room for individuality and solo accomplishment.