Skip to main content

Code Review

Status: 🚧 Stub

How we review code at WDS — both the mechanics and the culture.

Topics to cover​

  • What every PR should include before review (description, screenshots, test plan, etc.)
  • The reviewer's checklist (correctness, security, performance, accessibility, standards)
  • Required vs. optional reviewers — who needs to approve what
  • Expected turnaround time for reviews
  • How to give feedback (tone, suggestions, blocking vs. non-blocking comments)
  • How to respond to feedback as the author
  • When to pair on a review instead of doing it async
  • Escalation when reviewer and author disagree

Owner: TBD | Last reviewed: TBD