Goldilocks And The Three Code Reviews
summary
A practical postmortem: a big merged PR causes staging to crash because a duplicate controller slips through review, prompting a ~500 responses survey that finds reviews make devs better (avg 8.6/10, Ruby ~9.2), most reviews take about 5–20 minutes, and historical methods show inspections catch ~60% of defects while short reviews cut error rates from 55% to 2%. Recommendations are concrete: use linters and PR templates, include screenshots and repro steps, review everyone, be concise and kind, and iterate on the process.
AI-written summary · about
chapters (8)
- 0:00Start
- 0:12Goldilocks and the Three Code Reviews
- 12:21defect-detection
- 14:54working meetings
- 22:39big commits, large PR's, no context
- 29:232. concise w/ context
- 32:17start the conversation
- 32:35bettercode.reviews
related talks
47mGoldilocks and the Three Code Reviews
42mGoldilocks And The Three Code Reviews
47mGoldilocks and the Three Code Reviews
45mCracking the Code Review
32mLooks GREAT To Me: Getting Past Bare Minimum Code Reviews
31mMay the Code Review Be with You
23mDrama-Free Code Review!
32m👍 Looks GREAT To Me: Getting Past Bare Minimum Code Reviews
