| id | DEV-365 | |
|---|---|---|
| title | Mark a PR Ready Only When It Is Complete | |
| status | active | |
| enforcement | manual | |
| severity | error | |
| depends_on |
|
A PR marked ready before it has a reviewer, a preview, or passing CI wastes reviewer time on unfinished work and can merge unchecked.
Treat "ready for review" as a promise that the PR is complete and only the review remains. Prepare everything first, then flip it, so a reviewer never opens half-finished work.
- Assign at least one reviewer.
- Include a preview link when the change is visually verifiable.
- Mark ready for review only once the PR is a linked, self-assigned draft (per DEV-360) with a reviewer and a preview where applicable.
- Resolve all CI checks. CI runs after the PR is marked ready, so do not request approval until it passes.
- Do not merge without an approved review and passing CI.
- The PR has at least one reviewer and a preview link where applicable
- It is marked ready only after it is fully prepared
- CI passes before approval is requested
- It is not merged without an approved review and passing CI