Skip to content

Test exception catching in test_missing_file - #9874

Merged
pllim merged 2 commits into
astropy:masterfrom
astrofrog:fix-test-missing-file
Jan 21, 2020
Merged

pllim merged 2 commits into
astropy:masterfrom
astrofrog:fix-test-missing-file

Conversation

@astrofrog

Copy link
Copy Markdown
Member

I think this must be due to an update in pytest.

Fixes #9872

@astrofrog

Copy link
Copy Markdown
Member Author

@pllim I've tagged you for review since you are a pytest exception/warning catching expert! 😉

@astrofrog

Copy link
Copy Markdown
Member Author

The Matplotlib dev failure is unrelated

… fact that plt.xlabel and plt.ylabel now pass loc=None by default to set_xlabel and set_ylabel.
Comment thread astropy/visualization/wcsaxes/core.py Resolved
@pllim

pllim commented Jan 21, 2020

Copy link
Copy Markdown
Member

Looks uncontroversial enough but I'll wait for the CI to finish before approving. Thanks!

@pllim pllim left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

32-bit/parallel passed and the Windows failure is being addressed at #9871, so approving and merging. Thanks!

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.

Parallel build failure on CircleCI

2 participants