-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Doc 152 changes #6679
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
Doc 152 changes #6679
Conversation
- correct obvious redirects - remove some dead links
08e3401
to
7c22939
Compare
We no longer use the original buildbot, remove the links.
|
||
* Acanthostega | ||
* Adrien F Vincent | ||
* Adrien F. VINCENT | ||
* Adrien VINCENT |
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.
Looks like the .mailmap file needs an update also see danhickstein below
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 don't know how the list is generated, but if you're planning to alter it by hand to remove the duplicated entrie then “Adrien F. Vincent” is fine instead of the 4 duplicated entries due.
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.
@afvincent Thanks the way we normally handle this is via the .mailmap file which git uses to generate the logs which we use via the python api
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.
Specifically this file https://github.com/matplotlib/matplotlib/blob/master/.mailmap
No description provided.