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

Fork of rhodecode-enterprise-ce

auth: don't break hashing in case of user with empty password....
auth: don't break hashing in case of user with empty password. In some cases such as LDAP user created via external scripts users might set the passwords to empty. The hashing uses the md5(password_hash) to store reference to detect password changes and forbid using the same password. In case of pure LDAP users this is not valid, and we shouldn't raise Errors in such case. This change makes it work for empty passwords now.
dan - - Load All Authors

File last commit:

r552:9a0f45b0 default
r2203:8a18c3c3 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/