~/talk

Continuous Improvements of The Code Review Process

Talkat GeeCON 2024 · 2024-05-15..05-17· [developer-experience]
linkedin:keren-kenzi
slides:none on file

Avoid giant changes like a 100-file PR and prefer small commits — she shows a real before/after where a 3-file PR cuts ping‑pong and merge conflicts. Add a pull-request template with ticket link, description, before/after screenshots and a “unit test added” checkbox; set rules (e.g. reviewers required, PR awaiting time 5 hours up to 1 day), use “ignore spaces” diffs or IDE compare, adopt a PR pool and async picking, leverage AI linters for initial feedback, and schedule reviews to avoid a 25-minute context switch.

AI-written summary · about

  1. 0:00Introduction and icebreaker
  2. 4:22Breaking tasks into smaller PRs
  3. 7:13Providing PR context and templates
  4. 8:17Setting PR rules and expectations
  5. 10:30Assigning PRs and avoiding bystander effect
  6. 12:20Balancing code review workload
  7. 14:06Being kind in code reviews
  8. 16:35PRs as learning opportunities
  9. 17:39Q&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.