u/pc/rhodecode-enterprise-ce-fork-pc Files · docs/collaboration/pull-requests.rst

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.

marcink - - Load All Authors

File last commit:

r1:854a839a default


                r2203:8a18c3c3

default

Download file

             pull-requests.rst
        
                    29 lines
            
             | 714 B
            
                | text/x-rst
            
             |
                RstLexer
            
             / docs / collaboration / pull-requests.rst
          
                    History
                
                 |
                  Source
                 | Raw
                 |Copy content
                 |Copy permalink

        marcink
    
project: added all source files and assets

              r1
            
      .. _pull-requests-ref:

      Pull Requests

      =============

      Once a |pr| is open, you can use a variety of different code review functions

      to ensure the quality or style of code is appropriate for your standards. The

      |pr| is then merged with the main branch when it is approved. The following

      steps are an overview of the workflow:

      1. Create a fork or branch of a |repo|.

      2. Complete your works and open a pull request.

      3. Notify users to review and approve the |pr|.

      4. Merge the code to the target |repo|.

      .. toctree::

          pr-flow

          pull-req-mgmt

          open-pull-request.rst

          review-pull-request.rst

          update-pull-request

          approve-pull-request.rst

          merge-pull-request.rst

          fixing-common-commits.rst

	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	.. _pull-requests-ref:

		Pull Requests
		=============

		Once a \|pr\| is open, you can use a variety of different code review functions
		to ensure the quality or style of code is appropriate for your standards. The
		\|pr\| is then merged with the main branch when it is approved. The following
		steps are an overview of the workflow:

		1. Create a fork or branch of a \|repo\|.
		2. Complete your works and open a pull request.
		3. Notify users to review and approve the \|pr\|.
		4. Merge the code to the target \|repo\|.

		.. toctree::

		pr-flow
		pull-req-mgmt
		open-pull-request.rst
		review-pull-request.rst
		update-pull-request
		approve-pull-request.rst
		merge-pull-request.rst
		fixing-common-commits.rst