|
|
|
|
|
by yjftsjthsd-h
1639 days ago
|
|
Yeah, > That's how it happens: a tiny little change flings the door wide open. Someone solves their own local problem and misses the bigger picture. feels like an opening to say "...which is why we mandate code review to try and prevent situations where a single person's lack of perspective gets shipped." Of course, that's banking on the reviewer(s) having a broader perspective, so it's only a probabilistic mitigation, but it beats nothing. |
|
Code review only works well to prevent this sort of thing if you have designated owners for each library/feature, who are on the hook for the robustness/security/privacy thereof, and whose signoff is mandatory.