~/talk

Goldilocks And The Three Code Reviews

Talkat RailsConf 2017 · 2017-04-25..04-27· [developer-experience, testing]
slides:none on file

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

  1. 0:00Start
  2. 0:12Goldilocks and the Three Code Reviews
  3. 12:21defect-detection
  4. 14:54working meetings
  5. 22:39big commits, large PR's, no context
  6. 29:232. concise w/ context
  7. 32:17start the conversation
  8. 32:35bettercode.reviews
~/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.