rhodecode-enterprise-ce Files · docs/collaboration/review-pull-request.rst

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

Download file

             review-pull-request.rst
        
                    27 lines
            
             | 850 B
            
                | text/x-rst
            
             |
                RstLexer
            
             / docs / collaboration / review-pull-request.rst
          
                    History
                
                 |
                  Source
                 | Raw
                 |Copy content
                 |Copy permalink

        marcink
    
project: added all source files and assets

              r1
            
      .. _review-requests-ref:

      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

        marcink
    
docs: added SAML documentation....

              r3290
            
      automatically if |RCE| detects that it can do so safely. You will see this

        marcink
    
project: added all source files and assets

              r1
            
      message: `This pull request can be automatically merged.`

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

      |pr|.

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages

marcink project: added all source files and assets	r1	.. _review-requests-ref:

		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
marcink docs: added SAML documentation....	r3290	automatically if \|RCE\| detects that it can do so safely. You will see this
marcink project: added all source files and assets	r1	message: `This pull request can be automatically merged.`

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