-
Notifications
You must be signed in to change notification settings - Fork 108
Reviewers
Tom edited this page Jun 13, 2023
·
1 revision
Reviewers are proven contributors who are trusted to help with PRs. Currently they are
- @flend
- @zenzombie
The main objective is to avoid old PRs getting stuck with no attention. Reviewing and commenting on code, marking as approved, requesting changes, and asking the author further questions are all useful.
Reviewers have merge rights, but should seek agreement from other reviewers or me before merging. (Use squash & merge, except when the author has made an effort on a multi-commit history, where you should use create a merge commit.)
They should not directly push to any branch on the main repo.