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

Fork of rhodecode-enterprise-ce

user-groups: fix potential problem with group sync of external plugins....
user-groups: fix potential problem with group sync of external plugins. - when using external plugin we used to check for a parameter that set the sync mode. The problem is we only checked if the flag was there. So toggling sync on and off set the value and then left the key still set but with None. This confused the sync and thought the group should be synced !
dan - - Load All Authors

File last commit:

r552:9a0f45b0 default
r2143:4314e88b 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/