~/talk

Effective Code Reviews

slides:none on file

Code reviews rarely catch deep logical bugs for typical web apps; a Microsoft 2012 study shows their biggest wins are code understanding, small improvements and better team communication, while companies like Apple use specialist review processes to hunt critical defects. Use pull requests or Crucible for incremental reviews, prefer trunk-based work with small commits, require CI + unit tests, limit group reviews to seven people, have someone unfamiliar explain the code, keep a short checklist (5–10 items) and open follow-up tickets. leave your ego at the door

AI-written summary · about

  1. 0:00Intro
  2. 1:03Wait, why are you here?
  3. 1:45The good news...
  4. 3:57Developer Ego
  5. 5:11Ofc, it's your choice...
  6. 6:21What is your intention?
  7. 7:36Main Motivation for reviews?
  8. 9:37Are you tracking your Results?
  9. 11:48GOOD LUCK WITH THAT
  10. 12:14Leave your ego at the door
  11. 13:59Forget about "your" code
  12. 14:34Team reviews are a good start
  13. 17:41The intention matters
  14. 18:28Understanding is important
  15. 19:48Pick the right method...
  16. 23:21Create a coding guideline
  17. 25:34Keep track of results
  18. 27:56Create a review checklist
  19. 30:02Make your reviews effective!
  20. 31:17No matter what...
  21. 32:52Questions? Feedback?
~/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.