pagure-dist-git

#124 Add support for the collaborator access level/push
Merged by nphilipp. Opened by pingou.
collaborator_access  into  master

Download 124.patch

Until this commit we only allowed project committer to push to the
project on dist-git while pagure has gained a few months back a new
access level: collaborator which allow to give some user access to only
certain branches.

With this commit we're turning on this feature on dist-git as well.

Fixes https://pagure.io/releng/issue/9834

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr

Metadata Update from @nphilipp:
- Request assigned

For consistency, maybe rename is_committer to is_collaborator or may_commit?

Looks good to me!

rebased onto 2e491c9613b392234f900a2f721ee3bf2d53165c

Shouldn't the debugging messages here and above reflect the true meaning? E.g. "May commit:" above and "Push by committer/collaborator"? Not sure how important clarity is for debugging output like this.

As discussed on IRC, the exact wording of debug logs is not terribly important, more that you can follow what happened and why with the logs. Merging.

Pull-Request has been merged by nphilipp
Metadata
nphilipp
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.