In order to give credit to everybody involved in a commit I suggest a change to the commit message format (incl. its corresponding hook).
This way we can give credit to the people that reported an issue, did conceptual work, designed things, mentored a new contributor and more.
Opposed to other OSS projects I’d like to advise against adding a percentage to the amount of contribution. First off it just wastes time and secondly every contribution counts, no matter how insignificant it might seem.
New Commit Message Format proposal
[BreakingIndicator][PatchTag] Title Verbose message about the change. * Bullet points are okay, too * An asterisk is used for the bullet, it can be preceded by a single space. This format is rendered correctly by Forge (redmine) * Use a hanging indent Resolves: #IssueNumber Related: #IssueNumber Releases: [ReleaseTags] Reporter: Jane Doe Concept: Rachel Foucard UX: Benji Kott GFX: Luis Pato GFX: Marco Christian Krenn Mentor: - Proofread: Tom Warwick Reviewer: Oliver Klee Reviewer: Rens Admiraal Change-Id: I<some string generated by the git commit-msg hook>
As you can see, lines can exist multiple times in case more people participated in the patch.
I’m not 100% sure about the Reviewer thing, because Gerrit should take care of that.
- Adjust ContributionGuide
- Adjust pre-commit hook
- before merging an “old” patchset the merging person will ask around to find people involved in the patch. Yes, it’s overhead, but I’d argue it’s worth it.
- Show some respect and give credit where credit is due
- Time spent finding people who were involved.
Remarks and notes
Add everything from the discussion that does not fit other categories.
Topic Initiator: Mathias Schreiber
Topic Mentor: Susanne Moog