Skip to content

Latest commit

 

History

History
32 lines (25 loc) · 1.15 KB

File metadata and controls

32 lines (25 loc) · 1.15 KB
id DEV-430
title Deliver Bug-Free Work; Review Is a Safety Check
status active
enforcement manual
severity error
depends_on
DEV-410

Problem

Treating reviewers as the QA team pushes defects downstream, slows everyone, and lets subjective debates block otherwise sound work.

Solution

Own the quality of what you ship instead of outsourcing it to review. A reviewer who has to find your bugs becomes a bottleneck, and a defect caught in review costs more to fix than one you catch before opening the PR.

Test and self-review the change until you believe it is bug-free, then request review. Treat the reviewer as a final safety check on work you already trust, not the QA pass that first exercises it. When feedback is subjective rather than an objective problem (see DEV-410), discuss and push back instead of absorbing every preference as a required change, so sound work does not stall on taste.

Acceptance Criteria

  • Work is delivered bug-free, not dependent on review to find defects
  • Review is treated as a final safety check, not QA
  • Subjective feedback is pushed back on rather than blocking the PR