#1301 setting to require all changes use PR's
Closed: Fixed 6 years ago Opened 7 years ago by kevin.

It would be nice for some projects if there was a setting 'require PRs'.
This would make it so that even people with commit on the project couldn't directly push changes, they would have to make a PR and get that merged.

This would be nice for projects like fedora-comps or fedora-kickstarts where we want all changes to use PR's and sometimes people forget or whatever and push directly.


Basically, this would be a pre-update hook that does some checks to see where is the push coming from (ie: pagure or gitolite/a remote user)

Just wondering if this would be a per-project setting, or branch specific?

Just wondering if this would be a per-project setting, or branch specific?

We can do both, either you specify a branch (or multiples) and they are the one
blocker or you don't and then the whole project is impacted :)

This ticket has actually been fixed in https://pagure.io/pagure/pull-request/2206 (with a global setting though, not per branch)

Metadata Update from @pingou:
- Issue close_status updated to: Fixed
- Issue status updated to: Closed (was: Open)

6 years ago

Login to comment on this ticket.

Metadata