-
Notifications
You must be signed in to change notification settings - Fork 5.1k
Closed
Labels
status: assigned to maintainerWelcome onboardWelcome onboardstatus: needs feedbackTell me more about itTell me more about it🎯 featureCategorizes as related to a new featureCategorizes as related to a new feature
Milestone
Description
Reference https://about.gitlab.com/2014/09/29/gitlab-flow/
Reference https://code.google.com/p/gerrit/
Gogs should have a feature of protected branch. Let say master is a protected branch, then:
- Nobody can push directly to
master - Everybody* can checkout/fork from
master - Everybody* submit a merge request / code review request for his/her new branch (features/fixes)
- Everybody* can comment/discuss the diff, preferable line-by-line
- Only those with "Reviewer" or "Supervisor" or "Senior" privilege can vote up/down on a merge request / code review
- A merge request can only be eligible to be merged to master when having at least [a number] 2 of up votes. This parameter should be defined in a config file, environment variable, or runtime argument, so that nobody, even master or supreme admin can bypass code review process by setting it to 1 and self-review their code.
*Everybody is any user with sufficient authorization to read a project master and write to their own name space.
BoboTiG, chrberger, Drizin, rgoomar, denghongcai and 65 moregrotesque, Techwolf12 and ljqsluedecke, alxlion, tooolbox, thibaultmeyer, talex004 and 5 more
Metadata
Metadata
Assignees
Labels
status: assigned to maintainerWelcome onboardWelcome onboardstatus: needs feedbackTell me more about itTell me more about it🎯 featureCategorizes as related to a new featureCategorizes as related to a new feature