rhodecode-enterprise-ce Files · docs/code-review/code-review.rst

pull-requests: increase stability of concurrent pull requests creation by flushing prematurly the statuses of commits....

pull-requests: increase stability of concurrent pull requests creation by flushing prematurly the statuses of commits. This is required to increase the versions on each concurrent call. Otherwise we could get into an integrity errors of commitsha+version+repo

marcink - - Load All Authors

File last commit:

r3290:ac4e4e5a default


                r3368:a4f559a8

default

Download file

             code-review.rst
        
                    27 lines
            
             | 1.0 KiB
            
                | text/x-rst
            
             |
                RstLexer
            
             / docs / code-review / code-review.rst
          
                    History
                
                 |
                  Source
                 | Raw
                 |Copy content
                 |Copy permalink

        marcink
    
project: added all source files and assets

              r1
            
      .. _code-review-ref:

      Code Review

      ===========

        marcink
    
docs: added SAML documentation....

              r3290
            
      |RCE| provides two ways in which you can review code. You can review |prs| or

        marcink
    
project: added all source files and assets

              r1
            
      commits. To better understand |prs|, see the :ref:`pull-requests-ref`

      and :ref:`collaborate-ref` sections. For more information about why

      code review matters, see these posts on the topic:

      * `Code Review - Fix Bugs Early and Often`_

      * `Code Review - How to Convince a Skeptic`_

      * `Code Review - Learn How NASA Codes`_

      You can also use the |RCE| API set up continuous integration servers to leave

      comments from a test suite. See the :ref:`api` and

        dan
    
docs: updated docs for integrations, fixes #4137...

              r552
            
      :ref:`extensions-hooks-ref` sections for examples on how to set this up.

        marcink
    
project: added all source files and assets

              r1
            
      .. toctree::

         review-diffs

         approve-changes

         reviewing-best-practices

      .. _Code Review - Fix Bugs Early and Often: https://rhodecode.com/blog/code-review-fix-bugs-early-often/

      .. _Code Review - How to Convince a Skeptic: https://rhodecode.com/blog/code-review-convince-skeptic/

      .. _Code Review - Learn How NASA Codes: https://rhodecode.com/blog/code-review-learn-nasa-codes/

	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	.. _code-review-ref:

		Code Review
		===========

marcink docs: added SAML documentation....	r3290	\|RCE\| provides two ways in which you can review code. You can review \|prs\| or
marcink project: added all source files and assets	r1	commits. To better understand \|prs\|, see the :ref:`pull-requests-ref`
		and :ref:`collaborate-ref` sections. For more information about why
		code review matters, see these posts on the topic:

		* `Code Review - Fix Bugs Early and Often`_
		* `Code Review - How to Convince a Skeptic`_
		* `Code Review - Learn How NASA Codes`_

		You can also use the \|RCE\| API set up continuous integration servers to leave
		comments from a test suite. See the :ref:`api` and
dan docs: updated docs for integrations, fixes #4137...	r552	:ref:`extensions-hooks-ref` sections for examples on how to set this up.
marcink project: added all source files and assets	r1
		.. toctree::

		review-diffs
		approve-changes
		reviewing-best-practices

		.. _Code Review - Fix Bugs Early and Often: https://rhodecode.com/blog/code-review-fix-bugs-early-often/
		.. _Code Review - How to Convince a Skeptic: https://rhodecode.com/blog/code-review-convince-skeptic/
		.. _Code Review - Learn How NASA Codes: https://rhodecode.com/blog/code-review-learn-nasa-codes/