Community managers handle the first part of the process of responding to pull requests, before they are reviewed by core committers. Community managers are responsible for monitoring the GitHub project so that they are aware of incoming pull requests. For each pull request, a community manager should:
At this point, the pull request is ready for code review. There are two different options: small PR review and large PR review. A PR is “small” if it can be read and understood in less than 15 minutes, including time spent context-switching, reading the description of the pull request, reading any necessary code context, etc. Typically, “small” PRs consist of fixing typos, improving documentation, adding comments, changing strings to unicode, marking strings that need to be translated, adding tests, and other chores. A “small” pull request doesn’t modify the code that will be run in production in any meaningful way.
If the pull request is small, it can be reviewed immediately. If the community manager that is handling this pull request feels comfortable doing the code review, then he or she should do so rather than handing it off to a core committer. If not, he or she should move the JIRA ticket for the PR review into the “Awaiting Prioritization” state and add enough detail on the ticket for the product team to understand the size and scope of the changes. Inform the author that it might take a few days for the engineering team to review the PR.
If the pull request is not small, it will be handled by the full pull request process:
The community manager should:
For determining which teams that the pull request impacts, use common sense – but in addition, there are a few guidelines:
Once the code review process has started, the community managers are also responsible for keeping the pull request unblocked during the review process. If a pull request has been waiting on a core committer for a few days, a community manager should remind the core committer to re-review the pull request. If a pull request has been waiting on a contributor for a few days, a community manager should add a comment to the pull request, informing the contributor that if they want the pull request merged, they need to address the review comments. If the contributor still has not responded after a few more days, a community manager should close the pull request. Note that if a contributor adds a comment saying something along the lines of “I can’t do this right now, but I’ll come back to it in X amount of time”, that’s fine, and the PR can remain open – but a community manager should come back after X amount of time, and if the PR still hasn’t been addressed, he or she should warn the contributor again.