Skip to content

add protected branch #3921

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 2 commits into from
Closed

add protected branch #3921

wants to merge 2 commits into from

Conversation

ningdb
Copy link

@ningdb ningdb commented Dec 1, 2016

Refer to gitlab to develop this functionality
Admin does not restrict
Owner is not restrict
write permission restrictions
fix #776

@strk
Copy link
Contributor

strk commented Dec 2, 2016

@ningdb would you consider sending a version of this PR rebased to gitea/master ?
See https://github.com/go-gitea/gitea

@unknwon
Copy link
Member

unknwon commented Dec 22, 2016

Hi @ningdb , thanks for the PR! But I do really want to consider @strk 's idea because this PR is most likely not going to be merged as #776 assigned to myself to prevent any community contribution before I implemented the initial version.

@unknwon unknwon added the status: review later The pull request is not confident to be review at the moment label Dec 22, 2016
@joelhy
Copy link

joelhy commented Dec 22, 2016

@unknwon
When will you implement the initial version?
#776 has been wanted since Dec 23, 2014. And lots of people are requesting for it.

@unknwon
Copy link
Member

unknwon commented Feb 17, 2017

Closed by 7e09d21.

@unknwon unknwon closed this Feb 17, 2017
Martchus pushed a commit to Martchus/gogs that referenced this pull request Aug 27, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status: review later The pull request is not confident to be review at the moment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants