-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Autogenerate credits.rst #11399
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
Autogenerate credits.rst #11399
Conversation
doc/users/credits.rst
Outdated
Ilya Flyamer, | ||
ImSoErgodic, | ||
Importance of Being Ernest, | ||
ImportanceOfBeingErnest, |
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.
@ImportanceOfBeingErnest which one of these do you prefer? (this just needs to be added as an entry in the .mailmap file)
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 guess the one without spaces is better, it makes it more obvious that this is a username, and not a user name.
doc/users/credits.rst
Outdated
Patrick Chen, | ||
Patrick Marsh, | ||
Paul, | ||
Paul Barret, | ||
Paul G, | ||
Paul Ganssle, |
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.
Paul G (just above) is Paul Ganssle, please add an entry to the mailmap (see e.g. git help shortlog).
doc/users/credits.rst
Outdated
Ben Cohen, | ||
Ben Gamari, | ||
Ben Keller, | ||
Ben Root, | ||
Benedikt Daurer, | ||
Benedikt J. Daurer, |
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.
mailmap this
doc/users/credits.rst
Outdated
Devashish Deshpande, | ||
Diego Mora Cespedes, | ||
Dietmar Schwertberger, | ||
DietmarSchwertberger, |
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.
mailmap
doc/users/credits.rst
Outdated
JulianCienfuegos, | ||
Julien L, |
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.
That's Julien Lhermitte (just below), see https://github.com/matplotlib/matplotlib/pull/8455/commits.
doc/users/credits.rst
Outdated
Matthew Brett, | ||
Matthew Emmett, | ||
Matthew Newville, |
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.
same as Matt Newville
doc/users/credits.rst
Outdated
Truong Pham, | ||
Tuan, | ||
Tuan Dung Tran, | ||
Tuan333, |
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.
The 3 "Tuan" should be mailmap'd.
doc/users/credits.rst
Outdated
Zac Hatfield-Dodds, | ||
Zac-HD, |
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.
mailmap with Zac Hatfield-Dodds
doc/users/credits.rst
Outdated
yinleon, | ||
zhangeugenia, | ||
zhoubecky, | ||
Élie Gouzien |
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.
Something based on locale.strcoll should sort this properly at "E".
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.
locale.strcoll
would need a sorted(cmp=)
which was recently removed. Tried with contributors.sort(key=locale.strxfrm)
but that does not work for some reason.
doc/users/credits.rst
Outdated
burrbull, | ||
butterw, | ||
cammil, | ||
captainwhippet, | ||
castany, | ||
cclauss, | ||
cgohlke, |
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.
Christoph Gohlke
doc/users/credits.rst
Outdated
marky, | ||
masamson, | ||
mattip, |
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.
Matti Picus
doc/users/credits.rst
Outdated
nwin, | ||
ob, | ||
patniharshit, |
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.
Harshit Patni
4c5512a
to
5f94e44
Compare
Thanks @timhoffm ! |
PR Summary
This adds a script that automatically generates
credits.rst
with all the contributors in the git repository.As a checkup, that the auto-generated
credits.rst
does not miss someone listed in the previously manually maintained list, I've regenerated the file. Usually, it will be to run the script once before a release.