Skip to content

Commit c2c4f45

Browse files
authored
docs: avoid PRs based on unmerged PRs (#68)
1 parent 36483ff commit c2c4f45

File tree

1 file changed

+9
-2
lines changed
  • docs/contributing/guides-for-maintainers

1 file changed

+9
-2
lines changed

docs/contributing/guides-for-maintainers/triage.md

+9-2
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,9 @@ A **Maintainer** will check the Pull Request for validity.
6868

6969
There are no priorities or severities applied to Pull Requests.
7070

71-
A valid Pull Request is on-topic, well-formatted, contains expected information, and does not violate the code of conduct.
71+
A valid Pull Request is on-topic, well-formatted, contains expected information, does not depend on an unmerged Pull Request, and does not violate the code of conduct.
72+
73+
A Draft Pull Request is ignored and not triaged.
7274

7375
## Title
7476

@@ -84,7 +86,12 @@ If the Pull Request is related to a previous RFC process, the RFC document is re
8486

8587
## Commit History
8688

87-
Pull Request branches from a recent `main` commit.
89+
Pull Request branches from a recent `main` commit.
90+
91+
Pull Request does not depend on another unmerged Pull Request.
92+
93+
> [!NOTE]
94+
> Pull Requests that depend on unmerged Pull Requests add unnecessary complexity to the review process: **Maintainers** must track the status of multiple Pull Requests and re-review them if the dependent Pull Request is updated. Such dependency is much easier for the Pull Request author to track and to submit the Pull Request after all dependent code is already merged to `main`.
8895
8996
It is OK if the commit history is messy. It will be "squashed" when merged.
9097

0 commit comments

Comments
 (0)