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

svn: fixed case of wrong extracted repository name for SSH backend. In cases...

svn: fixed case of wrong extracted repository name for SSH backend. In cases where we commited to a nested subdirs SVN reported the access path with the subdir paths in it. We couldn't then match that extended name into proper rhodecode repository for ACL checks. - Current implementation gives an slight overhead as we have to lookup all repositories - fixes

marcink - - Load All Authors

File last commit:

r3290:ac4e4e5a default


                r4281:5da17e74

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\|.