Skip to content

Commit 23d0ec4

Browse files
gigaherzlearn-more
authored andcommitted
Add a document on the rules we should follow while managing/reviewing PRs,
and link to it from CONTRIBUTING.md and CODE_OF_CONDUCT.md
1 parent 994eb59 commit 23d0ec4

File tree

3 files changed

+24
-0
lines changed

3 files changed

+24
-0
lines changed

CODE_OF_CONDUCT.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,7 @@
11
# Code of Conduct
22

33
Be respectful toward other members of the community. Don't initiate or engage in discussions that are designed to insult, inflame, attack, or incite hate/discrimination/bullying against others. Don't feed the trolls. If we detect any kind of behaviour we consider unacceptable, we reserve the right to ban, block, or remove the involved members from any or all of our communities.
4+
5+
# See Also
6+
7+
- [Rules for managing Pull Requests](PULL_REQUEST_MANAGEMENT.md)

CONTRIBUTING.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -101,3 +101,7 @@ Finding a good project to start with can be a challenge, because when starting o
101101
[Coverity]: https://scan.coverity.com/projects/reactos
102102
[request-coverity]: https://scan.coverity.com/memberships/new?project_id=reactos
103103
[commit template]: .gitmessage
104+
105+
# See Also
106+
107+
- [Rules for managing Pull Requests](PULL_REQUEST_MANAGEMENT.md)

PULL_REQUEST_MANAGEMENT.md

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,16 @@
1+
# Rules for managing Pull Requests
2+
3+
For the sake of trying to maintain an acceptable number of open but idle PRs, the following rules should be considered:
4+
- If a PR has at least one approval, it can be merged after 1 week of waiting for additional comments.
5+
- If the change has at least 3 approvals or you consider it trivial enough, it may be merged right away.
6+
- If a PR stays in "changes requested" for too long, and there is no indication from the author that they are working on it, it shall be closed.
7+
- Rule of thumb: 2 weeks for a small PR. Can be longer if the PR is large.
8+
- The PR can be reopened at any point, if you have additional comments, or new changes have been done.
9+
- If you require a review from a particular person, assign the PR to that person. Don't just rely on the "review requested" feature of GitHub.
10+
- Remember that PR labels exist. You can assign an appropriate label to a pull request to designate it's scope, grab additional attention or just for extra navigation possibilities.
11+
- Don't feel obliged to comment everything you see, just for the sake of commenting. Be it on JIRA, GitHub, or even on IRC.
12+
13+
In addition, in order to avoid coming off as rude to helpful contributors, please refrain from:
14+
- Asking the contributor to do unrelated work
15+
- Closing without providing a reason
16+
- Merging with the intention to rewrite that code soon after

0 commit comments

Comments
 (0)