html_url,issue_url,id,node_id,user,created_at,updated_at,author_association,body,reactions,performed_via_github_app,issue
https://github.com/pydata/xarray/pull/6871#issuecomment-1204463757,https://api.github.com/repos/pydata/xarray/issues/6871,1204463757,IC_kwDOAMm_X85HyqiN,14371165,2022-08-03T20:44:15Z,2022-08-03T20:45:33Z,MEMBER,"```
_________________ ERROR at teardown of test_assert_valid_xy ___________________
[gw1] linux -- Python 3.8.13 /home/runner/micromamba/envs/xarray-tests/bin/python

    @pytest.fixture(scope=""function"", autouse=True)
    def test_all_figures_closed():
        """"""meta-test to ensure all figures are closed at the end of a test
    
        Notes:  Scope is kept to module (only invoke this function once per test
        module) else tests cannot be run in parallel (locally). Disadvantage: only
        catches one open figure per run. May still give a false positive if tests
        are run in parallel.
        """"""
        yield None
    
>       open_figs = len(plt.get_fignums())
E       NameError: name 'plt' is not defined
```
Hmm, strange error.  _assert_valid_xy has no matplotlib stuff so there should be no need to make sure all figures are closed. Oh well just adding the requires_matlplotlib solves this.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1326509734
https://github.com/pydata/xarray/pull/6871#issuecomment-1204087274,https://api.github.com/repos/pydata/xarray/issues/6871,1204087274,IC_kwDOAMm_X85HxOnq,14371165,2022-08-03T15:15:59Z,2022-08-03T20:32:46Z,MEMBER,"I was thinking that mypy failing is sufficient testing.

Edit: Was thinking of another PR. Added a test. :)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1326509734