##// END OF EJS Templates
u » ewong » rhodecode-enterprise-ce-fork
RSS

Fork of rhodecode-enterprise-ce

vcs: Use a thread scoped cache invalidation context to cache repository objects....
vcs: Use a thread scoped cache invalidation context to cache repository objects. Without this change the cache is on a process scope. If running with multiple threads this leads to sharing the cached object between threads. This will cause exceptions if multiple threads are trying to access the same curl object. Even worse it allows multiple threads to operate on the same repository object concurrently.
dan - - Load All Authors

File last commit:

r552:9a0f45b0 default
r614:cbe55781 default
Show More
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
===========
|RCM| provides two ways in which you can review code. You can review |prs| or
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/