~/talk

Code Reviews - Why, what and how

Talkat Meeting C++ 2017 · 2017-11-09..11-11· [engineering-leadership]
github:arnemertz
slides:none on file

Shows empirical claims and concrete practices: historical studies report 20–30% bug reduction from inspections, automated tests overlap but don’t replace reviews, and edge cases and naming still need humans. Recommends tooling like clang-format and CI, measuring coverage and profiling for performance, and process rules: open small digestible pull requests, include a checklist/pull-request template, run local tests before opening, review promptly, prioritize bugs over nitpicks, and prefer solo first-pass reviews then short paired walkthroughs for major features.

AI-written summary · about

  1. 0:00Introduction and goals
  2. 1:00Why do code reviews?
  3. 2:33QA and bug reduction
  4. 5:01Code reviews vs automation
  5. 6:40Performance and efficiency
  6. 7:55Code quality and style
  7. 8:38Learning and ownership
  8. 12:30What to review
  9. 15:17Focusing on what matters
  10. 17:26Documentation and context
  11. 20:05Process and culture
  12. 22:22Preparing for review
  13. 24:37Avoiding large requests
  14. 26:48Effective communication
  15. 28:43Reviewer best practices
  16. 31:19The review workflow
  17. 33:24Structural review steps
  18. 34:51What to look for
  19. 36:27Constructive feedback
  20. 38:33Discussion methods
  21. 39:35Reviewing the code, not user
  22. 41:01Finalizing the review
  23. 44:01Reviewer selection
  24. 46:28Team sharing
  25. 47:18Iterating the process
  26. 48:34Q&A session
~/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.