kou commented on PR #13184:
URL: https://github.com/apache/arrow/pull/13184#issuecomment-1139308026
> Looks `committer` becomes `Github` if we merge patches with this new
script.
>
> ```
> $ git shortlog -csn HEAD^..HEAD
> 1 GitHub
> ```
Good catch!
It seems that we should use `--group=trailer:signed-off-by` instead of `-c`
because we use commiter information for `Signed-off-by` in the merge script.
```console
$ git shortlog -csn 'HEAD^..HEAD'
1 GitHub
$ git shortlog -sn --group=trailer:signed-off-by 'HEAD^..HEAD'
1 Sutou Kouhei
``
> We use the command to collect committers in release notes, maybe it's an
issue need to address. There are `5 GitHub` in 8.0 releae notes
https://arrow.apache.org/release/8.0.0.html.
It seems that the `GitHub` was caused by pull requests merged with the
"merge" button.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
