3 ms·
> Code reviews have become a social thing, and bagging on someone with a litany of issues in front of a group is never acceptable behavior. If they've done ten
by majormajor 6y ago
> Code reviews have become a social thing, and bagging on someone with a litany of issues in front of a group is never acceptable behavior. If they've done ten things wrong, you have to pick 3 of them and let the rest go, or decide to kick them out of the group entirely.
You could talk to the person directly and point out those issues without dragging them in front of the group.
When there's a LOT wrong it usually indicates that their mental model isn't quite correct, and that is gonna be hard to correct without a conversation anyway.