the.com/code review
where your elegant solution meets someone asking why you didn't just use a hashmap.
means the process of a colleague reading your code before it ships, to catch bugs, share knowledge, and gently question your life choices.
from formalized in the 1970s as fagan inspection at ibm, an exhaustive line-by-line audit process, later loosened into the casual pull-request comments we know today.
bug catch ratefinds far more defects than testing alone
fagan method1976, ibm, absurdly rigorous by modern standards
real purposeknowledge transfer matters more than bug hunting
optimal lengthreviews over 400 lines lose effectiveness fast