~/talk

Code review beyond code style

Talkat Laracon EU 2017 · on Tuesday, 2017-08-29· [developer-experience, testing]
slides:none on file

Argues that code review focused on concrete practices — Git-based PRs, atomic commits, test‑first workflow with PHPUnit, and CI checks via StyleCI/Travis — makes reviews faster, reduces style arguments and surface bugs before deploys. Recommends Git add -p, keepachangelog.com for PR descriptions, GitHub review flow, and a docs repo or GitBook for rules. Uses the sandwich and suggestive language techniques to avoid interpersonal blowups. Teams report higher maintainability, fewer style debates, but occasional micromanagement needs offline resolution.

AI-written summary · about

  1. 0:00Start
  2. 1:19Code Reviews beyond Code Style
  3. 10:06Atomic Commits
  4. 15:15Check Your Code Duplication
  5. 16:49Things That You Should Do as a Reviewer
  6. 17:45Design Patterns
  7. 27:19Reptile Brain
  8. 28:27Use Suggestive Language
  9. 33:36Who Should Click this Button
  10. 41:01Git Book
~/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.