Agenda

  1. Warm-up: Read the Diff — a change to shared code, and the question its author is hoping somebody asks
  2. Team build period: features on branches, merged only after review
  3. The Code Review — a structured review of a teammate’s code, run with the protocol that keeps it about the code
  4. Merge, and record in the history what each review actually changed

Things to do before our next class

  • Milestone: every team member has reviewed one teammate’s code and has had their own code reviewed.
  • Journal: one review comment you received that improved the program, quoted exactly as it was written.