Add tutorial guide for getting your merge request reviewed
Summary
The existing contributor documentation has three separate pages that discuss getting your merge request reviewed but these sections have not been updated in quite some time and these may not be the most helpful resources for community contributors going through the review process.
- @aqualls has put together a new tutorial for how to do a merge request review (gitlab-org/gitlab!148910 (merged)) which could be a good resource for GitLab reviewers & MR coaches to use.
- We discussed whether it might be helpful to have a tutorial for getting your merge request reviewed / going through ping-pong with a reviewer.
- This would give GitLab a tutorial on getting reviewed and a tutorial on giving a review.
Existing contributor documentation on getting reviewed
- Tutorial -> Create a merge request -> Getting a review: https://docs.gitlab.com/ee/development/contributing/#review-process
- This section mentions
@gitlab-bot
commands and workflowready for review but doesn't get into the details of receiving feedback from a reviewer or how to go back and forth with a reviewer
- This section mentions
- Development page -> Review process: https://docs.gitlab.com/ee/development/contributing/#review-process
- This section goes into more details on the review process from GitLab's side and offers some tips for the contributor ahead of asking for a review. But still doesn't get into how to respond to a reviewer's comments or give clear step by step instructions on getting through a successful review process.
- Code Review Guidelines: https://docs.gitlab.com/ee/development/code_review.html
- This document is quite overwhelming especially for first time contributors to GitLab. It covers a lot of information that is not all critical for a community contributor to know and there is little structure here in walking through what to do. It is also from the perspective of a team member reviewing and not from a community contributor getting reviewed.