This document is a guideline for how core contributors should review and merge pull requests (PRs). It is intended to outline the lightweight process that we’ll use for now. It’s assumed that we’ll operate on good faith for now in situations for which process is not specified.
PRs may only be merged after the following criteria are met:
- It has been open for at least 1 business day
- It has no
vetolabel on it - It has been 'LGTM'-ed by 3 different reviewers, each from a different organization
- It has all appropriate corresponding documentation and testcases
When a reviewer deems a PR good enough to merge, they should add a comment to the PR thread that simply reads 'LGTM'. If they do not deem it ready for merge, they should add comments -- either inline or on the PR thread -- that indicate changes they believe should be made before merge.
If a reviewer decides that a PR should not be merged in its current state,
even if it gets 3 'LGTM' approvals from others, they should mark that PR with
veto label.
This label should only be used by a reviewer when that person believes there is a fundamental problem with the PR. The reviewer should summarize that problem in the PR comments and a longer discussion may be required.
We expect this label to be used infrequently.