~/talk

Code Review vs Pul Requests

Talkat Devoxx US 2017 · 2017-03-21..03-23· [developer-experience]
links:none on file
slides:none on file

Compares four concrete review workflows and what each actually enforces: git format-patch mailing lists give clean single commits and scale (Linux sees ~13,000 patches per release) but make iteration awkward; Crucible does post‑commit reviews across repositories with percent reviewed, due dates and time tracking but requires manual merges; Gerrit demands a single commit per review and encourages squash and force-push for a linear history; Bitbucket pull requests are strict branch‑to‑branch gates with CI and merge checks and multiple merge strategies (no‑ff, fast‑forward, squash).

AI-written summary · about

  1. 0:00Introduction
  2. 0:47History of Code Review
  3. 3:25Precommit Code Review
  4. 5:38Mailing List Workflow
  5. 6:20Single Commit
  6. 6:47Mailing Lists
  7. 7:38Pros and Cons
  8. 14:28Garrett
  9. 16:05Commits
  10. 17:45Pul Requests
  11. 20:07Pull Requests
  12. 26:44What to use
  13. 27:59Summary
  14. 28:41Who should use which
  15. 30:17Questions
~/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.