FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Bugfix/legend windowcall by WeatherGod · Pull Request #32 · matplotlib/matplotlib · GitHub

Repository navigation

Bugfix/legend windowcall - #32

Merged
1 commit merged into
matplotlib:v1.0.xfrom
WeatherGod:bugfix/legend_windowcall
Mar 7, 2011
Merged

1 commit merged into
matplotlib:v1.0.xfrom
WeatherGod:bugfix/legend_windowcall

Conversation

Copy link
Copy Markdown
Member

Fix the legend object's get_window_extent() call signature to match everyone else's get_window_extent() call signature. This will allow for legend() to be passed into bbox_extra_artists kwarg for savefig().

…ent with everyone else's get_window_extent()'s call signature.

mdboom commented Mar 7, 2011

Copy link
Copy Markdown
Member

Looks good to me.

Copy link
Copy Markdown
Member Author

Ok, so how do I do this properly? I haven't tried doing a merge into v1.0.x and then into master.

mdboom commented Mar 7, 2011

Copy link
Copy Markdown
Member

Not sure. I think Darren should know (and there are plans to add these instructions to the developer docs eventually)

jkseppan commented Mar 7, 2011

Copy link
Copy Markdown
Member

First make sure your v1.0.x branch is fresh:

git checkout v1.0.x
git pull --ff-only upstream v1.0.x

(This assumes an "upstream" remote pointing to git://github.com/matplotlib/matplotlib.git.) Then merge your pull request and run the test suite (I don't know if this should be mandatory):

git merge bugfix/legend_windowcall
python setup.py install # or however you compile things to run the tests
./tests.py # all new test runner script!

If everything is fine, push:

git push git@github.com:matplotlib/matplotlib.git v1.0.x

Next do the same with master, this time merging v1.0.x:

git checkout master
git pull --ff-only upstream master
git merge v1.0.x
python setup.py install
./tests.py
git push git@github.com:matplotlib/matplotlib.git master

I'll post these instructions on the developer list so people can comment on them.

richbwood mentioned this pull request Dec 19, 2012
WeatherGod deleted the bugfix/legend_windowcall branch July 23, 2014 12:55
This pull request was closed.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL