4 ms·
This - last year I broke something because I'd changed a query in a file completely unrelated to the ticket I was working on, and the guy who did the review tre
by jamesfinlayson 5y ago
This - last year I broke something because I'd changed a query in a file completely unrelated to the ticket I was working on, and the guy who did the review treated code review as a rubber-stamp - he completely ignored the fact I'd accidentally committed a change to payment processing code in a patch to tighten up company number validation.
- patrickwalton 5y agoEven if code reviewers only made sure the files changed made sense, the reviews would be worth it.