Skip to content

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

Merged
merged 6 commits into from
Jul 3, 2016
Merged

Doc 152 changes #6679

merged 6 commits into from
Jul 3, 2016

Conversation

tacaswell
Copy link
Member

No description provided.

@tacaswell tacaswell added this to the 1.5.2 (Critical bug fix release) milestone Jul 3, 2016
@tacaswell tacaswell mentioned this pull request Jul 3, 2016
We no longer use the original buildbot, remove the links.

* Acanthostega
* Adrien F Vincent
* Adrien F. VINCENT
* Adrien VINCENT
Copy link
Member

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

Copy link
Contributor

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.

Copy link
Member

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

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jenshnielsen jenshnielsen merged commit d978e02 into matplotlib:v1.5.x Jul 3, 2016
@tacaswell tacaswell deleted the doc_152_changes branch July 3, 2016 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants