~/talk

Goldilocks and the Three Code Reviews

Talkat RedDotRubyConf 2017 · 2017-06-22..06-23· [developer-experience]
slides:none on file

Research shows formal inspections catch ~60% of defects and cut defects by 20–30 per 1000 LOC, while short reviews drop error rates from ~55% to ~2% (others report 86% → 99.6% correctness). Surveys find most PRs get reviewed, reviewers spend ~5–20 minutes, and ~10% of teams require asking for reviews. Concrete fixes: add commit hooks, use GitHub PR templates and linters, make small PRs, assign reviewers, collect review metrics and iterate the process.

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.