~/talk

👍 Looks GREAT To Me: Getting Past Bare Minimum Code Reviews

Talkat DevOpsDays London 2024 · 2024-09-26..09-27· [developer-experience]
slides:none on file

Explains concrete fixes: reviewers leave ego behind, focus on code, run 25–45 minute focus bursts and refuse giant PRs; write objective, pinpointed comments using labels like needs change / needs rework / nitpick and the Triple R: request, rationale, result format; authors self-review, run linters/formatters/static analysis and tests, and make PRs atomic so they’re reviewable in 10–20 minutes — studies show reviews under ~500 lines and <20 files get more useful feedback; use PR templates and CODEOWNERS to auto-assign.

AI-written summary · about

  1. 0:00Introduction to LGTM problem
  2. 1:07What’s wrong with code reviews
  3. 7:10Fixing reviewer mistakes
  4. 10:19Writing effective comments
  5. 21:30Author’s common mistakes
  6. 25:25How authors can improve
  7. 29:37Going beyond bare minimum
~/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.