-
Notifications
You must be signed in to change notification settings - Fork 712
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 AUTHORS.md and .mailmap #501
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will send in a correction to the mailmap since this repo does not contain any of my older email addresses.
.mailmap
Outdated
Radovan Bast <radovan.bast@gmail.com> <radovan.bast@uit.no> | ||
Radovan Bast <radovan.bast@gmail.com> <rbast@users.noreply.github.com> | ||
Roberto Di Remigio <roberto.diremigio@gmail.com> <roberto.d.remigio@uit.no> | ||
Eric Noulard <eric.noulard@gmail.com> <enoulard@antidot.net> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Patchwork mail address is strange.
It would be better to have only my gmail address.
I am a bit concerned about email harversters, and suggest to leave the |
We can have it opt-in, sure. I generated this one looking at the log. I'll only keep bast@users.noreply.github.com for @bast and GMail for @TheErk What do you think? |
Thanks! From my perspective this would be great. Thank you. |
833cb56
to
f0b82b3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! Good for me. Also for you, Eric?
Good for me too. |
Does what it says on the tin. The list in
AUTHORS.md
was generated usinggit shortlog -sn
, so authors are ordered by number of commits. Which also happens to be alphabetic order on surnames!Status