Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork7.9k
Add tests for toolmanager.#16933
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
5448b1f
to1134670
CompareUh oh!
There was an error while loading.Please reload this page.
Restarted the failing CI job (sub-process timeouts on mac on travis). Anyone can merge on green. |
Subprocess still times out. |
I wouldn't be surprised if that really failed, but that'll need someone on a Mac to test locally... |
... and fix Help tool for tk (it needs a gif icon, generated (like theother gifs) with `convert -antialias -geometry 24x24 help.svg help.gif`).... and also edit the matplotlibrc template to mention its availability.
Actually toolmanager is clearly not implemented for macosx, so let's just skip it. |
Uh oh!
There was an error while loading.Please reload this page.
... and fix Help tool for tk (it needs a gif icon, generated (like the
other gifs) with
convert -antialias -geometry 24x24 help.svg help.gif
)(#6610).
... and also edit the matplotlibrc template to mention its availability.
Edit: plenty of new coverage:https://codecov.io/gh/matplotlib/matplotlib/compare/d9b722f8a6ee91955f83609b52b2d2b203316385...11346704f6e42f85b2a19cccc6f3be6b81451bb9/tree/lib/matplotlib
PR Summary
PR Checklist