#434: Contributing to a pagure repository
-------------------------------------+-------------------------------------
Reporter: robyduck | Owner:
Type: New badge | robyduck
idea | Status:
Priority: minor | reopened
Keywords: | Resolution:
Has a description: 1 | Has a name:
Artwork status: Approved | 1
(design team members only) | Concept approved (reviewers only):
External requirements: | 1
Triaged (triagers only): 1 | Badge definition status:
| Full, needs review
| Manually awarded:
| 0
-------------------------------------+-------------------------------------
Comment (by lsedlar):
The message format from Pagure has changed in the past I believe. I have
attached a patch that may fix the problem and only works with the current
format.
There are three places in the message with some name:
* {{{msg.agent}}} is the person who pushed the commits
* {{{msg.repo.user.name}}} is owner of the repo that was pushed to
* {{{msg.authors[].name}}} is a list of people who wrote the pushed
commits
My patch is using the first of these, as there there is no way to map
individual authors to commits they wrote. It will also accumulate the
number in msg.total_commits.
The big question is if this is the right approach: if the project is using
pull requests from individual forks, people will have to enable the fedmsg
hook for their forks (because merging via web interface does not send this
kind of message). If they do, they may get badges for pushes that never
make it into the main repo.
--
Ticket URL: <
https://fedorahosted.org/fedora-badges/ticket/434#comment:32>
fedora-badges <
https://badges.fedoraproject.org>
A place to collect and debate badge ideas for the Fedora Badges app