##// END OF EJS Templates
rhodecode-enterprise-ce
RSS
pull-requests: add merge check that detects WIP marker in title. This will prevent merges in such case....
pull-requests: add merge check that detects WIP marker in title. This will prevent merges in such case. Usually WIP in title means unfinished task that needs still some work. This pattern is present in Gitlab/Github and is already quite common.
marcink - - Load All Authors

File last commit:

r3290:ac4e4e5a default
r4099:c12e69d0 default
Show More
review-pull-request.rst
27 lines | 850 B | text/x-rst | RstLexer
/ docs / collaboration / review-pull-request.rst
History | Annotation | Raw |Copy content |Copy permalink

Review a |pr|

To review a pull request, use the following steps:

  1. Open the review request from :menuselection:`Admin --> Repositories --> repo name --> Pull Requests --> Awaiting my review`
  2. Leave your review comments inline, or in the commit message.
  3. Set the review status from one of the following options:
    • :guilabel:`Not Reviewed`
    • :guilabel:`Approved`
    • :guilabel:`Approved & Closed`
    • :guilabel:`Rejected`
    • :guilabel:`Rejected & Closed`
    • :guilabel:`Under Review`
  4. Select Comment

When the |pr| is approved by all reviewers you will be able to merge automatically if |RCE| detects that it can do so safely. You will see this message: This pull request can be automatically merged.

If rejected, you can fix the issues raised during review and then update the |pr|.