Skip to content

MNT: Special handling of matplotlib version checks for doc and test - #10956

Merged
dhomeier merged 3 commits into
astropy:masterfrom
pllim:mpl-special-bump
Oct 28, 2020
Merged

dhomeier merged 3 commits into
astropy:masterfrom
pllim:mpl-special-bump

Conversation

@pllim

@pllim pllim commented Oct 27, 2020

Copy link
Copy Markdown
Member

Description

This pull request is to address #10900 (comment) and fix #10954 .

  • Sphinx needs matplotlib >= 3.1 to build.
  • One test emits warning with matplotlib < 3.1.

@mhvk mhvk left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me!

Comment thread astropy/visualization/wcsaxes/tests/test_misc.py Outdated
to avoid PY_SSIZE_T_CLEAN warning with Python 3.9 and Matplotlib 3.0

Co-authored-by: Derek Homeier <[email protected]>
@dhomeier

dhomeier commented Oct 28, 2020 •

Copy link
Copy Markdown
Contributor

Haven't seen any request for changes (the conflict just arose from merging #10952); thanks for the fix!

@dhomeier
dhomeier merged commit 3b80208 into astropy:master Oct 28, 2020
@pllim

pllim commented Oct 28, 2020

Copy link
Copy Markdown
Member Author

Looks like there is a merge commit in the PR but it is already merged so I cannot rebase it out. @bsipocz , if this is an issue, please let me know.

@pllim
pllim deleted the mpl-special-bump branch October 28, 2020 18:45
@embray

embray commented Oct 30, 2020

Copy link
Copy Markdown
Member

Thanks, LGTM too.

bsipocz pushed a commit that referenced this pull request Nov 6, 2020
MNT: Special handling of matplotlib version checks for doc and test
bsipocz pushed a commit that referenced this pull request Nov 7, 2020
MNT: Special handling of matplotlib version checks for doc and test
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.

PY_SSIZE_T_CLEAN warning with Python 3.9 and Matplotlib 3.0

5 participants