~/talk

Looks GREAT To Me: Getting Past Bare Minimum Code Reviews

on YouTube 32m
Talkat Infobip Shift 2023 · 2023-09-18..09-19· [developer-experience, soft-skills]

speaker

name:Adrienne Braganza Tacke
website:https://looksgoodtomebook.dev

resources

video:https://youtu.be/1FdcIvfSi_A
slides:none on file

summary

Argues that “LGTM” often masks rushed reviews and gives concrete fixes: reviewers must use 25–45 minute review bursts, write objective comments with the Triple R pattern (request, rationale, result), and refuse oversized PRs; authors must run linting, static analysis and tests pre‑PR, make changes under 500 lines and split features so reviews finish in 10–20 minutes and under ~20 files. Recommends PR templates, code‑owners/auto‑assign and informational reviewers; ties improvements to DORA metrics like change‑failure rate.

AI-written summary · about

chapters (6)

  1. 0:00The problem with code reviews
  2. 2:31Common reviewer mistakes
  3. 10:14Common author mistakes
  4. 16:40How to give effective feedback
  5. 21:25Best practices for authors
  6. 27:36Team agreements and metrics

related talks

~/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.