~/talk

Goldilocks and the Three Code Reviews

Talkat Red Dot Ruby Conference 2017 · 2017-06-22..06-23· [developer-experience]
slides:none on file

Code reviews catch bugs earlier: formal inspections historically catch ~60% of defects and reduce defects 20–30% per 1,000 LOC, while short reviews drop error rates from ~55% to ~2% (or lift correctness from 86% to 99.6%). A 1–2 reviewer workflow often spends 5–20 minutes per PR; problems come from huge PRs, no context, and nitpicky comments. Practical fixes: commit hooks, GitHub PR templates, linters/formatters, small focused PRs, assigned reviewers, pairing, and iterating on the review checklist.

AI-written summary · about

  1. 0:00Start
  2. 8:34catching problems at the "lowest-value" stage
  3. 9:28collective ownership in construction
  4. 11:22inspections
  5. 13:49short reviews
  6. 14:50walk throughs
  7. 22:06Energy
  8. 24:33Substance
  9. 28:59SKYLIGHT
  10. 30:301. commit hooks
  11. 35:25I love code reviews in theory.
~/search
type to search 6,249 conferences, 138,987 speakers and 185,172 talks
~/help
cmdctrl k
search every conference, speaker and series
/
focus the filter search
h
home
c
conferences
s
speakers
e
series
t
light / dark — the status bar button does the same
?
this help
esc
close this, then clear every filter — or just the search, while you are typing in it

Ignored while you are typing. Pages without a filter rail have no search to focus.