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

Fix macOS CI test failure by QuLogic · Pull Request #17065 · matplotlib/matplotlib · GitHub

Repository navigation

Fix macOS CI test failure - #17065

Merged
efiring merged 1 commit into
matplotlib:masterfrom
QuLogic:fix-ci
Apr 8, 2020
Merged

efiring merged 1 commit into
matplotlib:masterfrom
QuLogic:fix-ci

Conversation

QuLogic commented Apr 8, 2020 •
edited
Loading

Copy link
Copy Markdown
Member

PR Summary

This updates the result PDF image for test_bbox_inches_tight_raster to work with Ghostscript 9.52. Somehow this also works fine with the old version too. I guess what we used to produce is just slightly different enough from what we do now, and new Ghostscript is also slightly different enough for both things together to break.

Fixes #17037.

PR Checklist

  • Has Pytest style unit tests
  • Code is Flake 8 compliant
  • [N/A] New features are documented, with examples if plot related
  • [N/A] Documentation is sphinx and numpydoc compliant
  • [N/A] Added an entry to doc/users/next_whats_new/ if major new feature (follow instructions in README.rst there)
  • [N/A] Documented in doc/api/api_changes.rst if API changed in a backward-incompatible way

QuLogic added this to the v3.2.2 milestone Apr 8, 2020
QuLogic mentioned this pull request Apr 8, 2020
6 tasks
efiring merged commit ce42e8e into matplotlib:master Apr 8, 2020
meeseeksmachine pushed a commit to meeseeksmachine/matplotlib that referenced this pull request Apr 8, 2020
tacaswell added a commit that referenced this pull request Apr 8, 2020
…065-on-v3.2.x

Backport PR #17065 on branch v3.2.x (Fix macOS CI test failure)
QuLogic deleted the fix-ci branch April 8, 2020 18:44
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Travis Failing in many PRs

2 participants


Back | FazBrowse Home | New Git URL