5 ms·
Reviewing someone's code you had not seen before, involving a third party vendor you know nothing about, that only happens in production environment and only ra
by coldcode 5y ago
Reviewing someone's code you had not seen before, involving a third party vendor you know nothing about, that only happens in production environment and only randomly fails and otherwise works—does not shock me at all. No process no matter how thorough can still fail. Code reviews in my prior experience solved nothing as people were expected to work full time and yet still review code continuously; unless you give engineers sufficient time to do both without overtime effort, one or the other will be affected, and usually its the reviews.
- mdaniel 5y agoI had always seen code review process as partially a knowledge sharing exercise, so fine, maybe none of the 3 other engineers had previously seen those vendor docs, but part of the process is to retrace the steps and see if (with the benefit of hindsight) the reviewer would reach the same conclusion > Code reviews in my prior experience solved nothing I find that assertion suspicious, unless the culture was "push the approval button without even reading because ship,ship,ship"