Re: [PATCH 0/8] parsing trailers with shortlog

From: Derrick Stolee <stolee@gmail.com>
Date: Sep 25, 2020, 14:27 UTC
Message-ID: <7e40593e-b41f-1c01-4005-5aa80d88302d@gmail.com>
In-Reply-To: <20200925070120.GA3669667@coredump.intra.peff.net>

On 9/25/2020 3:01 AM, Jeff King wrote:

Show 5 quoted lines

> Somebody mentioned at the inclusion summit that it would be nice for
> mentoring/pairing relationships if we could more easily give credit to
> multiple authors of a commit. When people ask about adding multiple
> "author" headers, we usually recommend that they use a "co-authored-by"
> trailer. But you can't convince shortlog to count it for anything. :)

This discussion was worthwhile, and could even have measurable effects on this community, depending on how Junio creates the list of contributors for the release notes.

Show 13 quoted lines

> So this series adds support for counting trailers to shortlog. You can
> do a number of fun things with it, like:
> 
>     # credit reviewers
>     git shortlog -ns --group=trailer:reviewed-by
> 
>     # credit authors and co-authors equally
>     git shortlog -ns --group=author \
>                      --group=trailer:co-authored-by
> 
>     # see who helps whom
>     git shortlog --format="...helped %an on %as" \
>                  --group=trailer:helped-by

I built Git with these patches and played around with these and other options (such as `--group=author --group=committer`). I could not find any bugs or other ways to improve these patches.

Thanks, -Stolee