Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Merge 2x to master #5936
Conversation
jenshnielsen
and others
added some commits
Jan 19, 2016
mdboom
added the
needs_review
label
Jan 28, 2016
|
so, most of these are already on v2.x, they just weren't marked as such? The diff seems rather paltry compared to the number of merge commits. |
QuLogic
added this to the
2.1 (next point release)
milestone
Jan 28, 2016
|
Presumably all those "merge" commits are just cherry-picked from master, so the merge back did nothing. |
WeatherGod
added a commit
that referenced
this pull request
Jan 28, 2016
|
|
WeatherGod |
1c5a45a
|
WeatherGod
merged commit 1c5a45a
into matplotlib:master
Jan 28, 2016
mdboom
removed the
needs_review
label
Jan 28, 2016
|
@QuLogic is right. The merge commits were backported and the new commits that creates will be empty when merged back to master |
jenshnielsen
deleted the
jenshnielsen:merge2xmaster branch
Feb 16, 2016
jenshnielsen
restored the
jenshnielsen:merge2xmaster branch
Mar 19, 2016
jenshnielsen
deleted the
jenshnielsen:merge2xmaster branch
Apr 2, 2016
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
jenshnielsen commentedJan 28, 2016
Yet another somewhat painful merge due to conflicts in pdf files so doing via PR