All changes are reviewed and the entire team is responsible for doing review.
The usual flow:
- Contributor a opens pull request.
- If they're still working on it, they preface the title with
[WIP](Work In Progress). - When it's ready for review, they comment on it saying so.
- Suddenly a wild reviewer appears!
- Reviewer assigns the PR to themselves.
- Reviewer leaves line comments with suggestions or questions.
- When the reviewer is done they comment on the PR with an emoji, meme, pokémon, or words.
- The contributor responds to feedback, makes changes, etc.
- When the contributor is ready for the reviewer to re-review, they comment on the PR with an emoji, meme, pokémon, or words.
- Goto 6 until both parties are happy with the PR.
- The reviewer merges the PR and deletes the branch.