Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add .mailmap with entries for top committers #2183

Merged
merged 1 commit into from
Mar 17, 2018

Conversation

cbeams
Copy link
Contributor

@cbeams cbeams commented Feb 26, 2018

This commit adds a .mailmap file as described in the git shortlog
manpage, normalizing names and email addresses for several top
committers.

For context, see the conversation between @cbeams and @harding at
https://twitter.com/cbeams/status/967491714335297536. Entries in a
.mailmap like those found in this commit eliminate the need to manually
"combine multiple entries" for authors like @wbinns who have had commits
with varying author metadata over time.

Ultimately, the purpose is to make the output of git shortlog as
concise and accurate as possible. Compare the results of git shortlog -nse before and after this commit to see the difference it makes.

This commit adds a .mailmap file as described in the `git shortlog`
manpage, normalizing names and email addresses for several top
committers.

For context, see the conversation between @cbeams and @harding at
https://twitter.com/cbeams/status/967491714335297536. Entries in a
.mailmap like those found in this commit eliminate the need to manually
"combine multiple entries" for authors like @wbinns who have had commits
with varying author metadata over time.

Ultimately, the purpose is to make the output of `git shortlog` as
concise and accurate as possible. Compare the results of `git shortlog
-nse` before and after this commit to see the difference it makes.
@wbnns wbnns self-assigned this Feb 27, 2018
@wbnns wbnns merged commit 0235b5d into bitcoin-dot-org:master Mar 17, 2018
@wbnns
Copy link
Contributor

wbnns commented Mar 17, 2018

@cbeams Thanks Chris.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants