-
-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Make all classes new-style. #3662
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
Conversation
The failures are not related. The 2.6 ones are fixed by #3661 and the other one is a random one. |
👍 If we rebase on master and give Travis another go... |
Sorry, I merged #3661 which apparently conflicts with this.. |
Does that mean I need to merge matplotlib/master into my branch to get Travis to rerun? |
Not that it is so important but it must be #3635 that conflicts with this one. |
We prefer that you do a re-base rather than merge master into your branch. This is done via
and follow the on-screen directions. 'upstream' is what ever name you have given to the 'matplotlbi/matplotlib' repository. The conflicts are in @jenshnielsen That makes more sense. |
sorry, forgot the last step, after you have cleaned up the conflicts and merged everything
|
e0a9377
to
f4e9c6c
Compare
I had to
And then it told me, can't --continue, there's nothing to resolve, so I did
The two changed lines that git considered in conflict because they were adjacent, are now both in their changed (better) states in switham/matplotlib. Maybe the weird middle part was because I had done a git commit --amend before the original pull request, so I had to re-merge the two duplicate commits again. I saved a screen log in case anyone's interested. Also, I don't see github proceeding to run Travis after I did this: (This has been edited, I mistakenly wrote "git commit --continue" instead of "git rebase --continue" before. Another edit later.) |
MNT : Make all classes new-style.
@switham Thanks! Is this your first contribution to mpl? |
This one and #3628 are my first two, yay! |
The theory is, the code already worked in Python 3 where all classes are new-style.
Changed class definitions in *.py (only) in the whole tree, including
Take it away, Travis.