Gitea: PR acceptance blocked for PR creator

Created on 19 Feb 2018  路  9Comments  路  Source: go-gitea/gitea

  • Gitea version (or commit ref): 1.2.3

Description

Our development team was thinking to use PR system to review our code, is there any feature to block the creator of the PR to accept it?

Screenshots

Image PR create
Image PR accept

On that example, the user administrator created de PR and accepted it.

kinfeature kinproposal

Most helpful comment

Right now, I don't think so. Probably only possible when #2794 is done. But correct me if I'm wrong.

All 9 comments

Right now, I don't think so. Probably only possible when #2794 is done. But correct me if I'm wrong.

I'm pretty sure that you can set a collaborator's permission to 'Read', and that should grant them ability to make PRs & such, but not push directly. Although, of course, a repo admin / someone with write access would be needed to merge the PR.

@lunny this is not resolved by #3689 - if the PR creator is on the whitelist they can merge. #2794 was abandoned. Is there any other issue in progress that would deliver this?

@cailyoung you can set that PR needs at least one approval

@lafriks 's idea is correct.

@lafriks @lunny

Apologies, we are not running 1.6.0 or newer. I will have a chat with the team here.

So #3748 fixes this - i.e. the PR creator cannot be an approver?

@cailyoung yes, one could not approve himself PR.

Was this page helpful?
0 / 5 - 0 ratings

Related issues

jorise7 picture jorise7  路  3Comments

BRMateus2 picture BRMateus2  路  3Comments

internalfx picture internalfx  路  3Comments

ghost picture ghost  路  3Comments

jonasfranz picture jonasfranz  路  3Comments