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/5474#issuecomment-865198998,https://api.github.com/repos/pydata/xarray/issues/5474,865198998,MDEyOklzc3VlQ29tbWVudDg2NTE5ODk5OA==,41898282,2021-06-21T17:04:02Z,2021-06-21T17:04:02Z,CONTRIBUTOR,"## Unit Test Results
0 files  ±0  0 suites  ±0   0s :stopwatch: ±0s
0 tests ±0  0 :heavy_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit 5381962c. ± Comparison against base commit 5381962c.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,922792799
https://github.com/pydata/xarray/pull/5474#issuecomment-862496239,https://api.github.com/repos/pydata/xarray/issues/5474,862496239,MDEyOklzc3VlQ29tbWVudDg2MjQ5NjIzOQ==,24736507,2021-06-16T15:53:06Z,2021-06-16T16:40:59Z,NONE,"Hello @dcherian! Thanks for updating this PR. We checked the lines you've touched for [PEP 8](https://www.python.org/dev/peps/pep-0008) issues, and found:

* In the file [`xarray/tests/test_coarsen.py`](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py):

> [Line 9:1](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L9): [F401](https://duckduckgo.com/?q=pep8%20F401) '.test_dataarray.da' imported but unused
> [Line 10:1](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L10): [F401](https://duckduckgo.com/?q=pep8%20F401) '.test_dataset.ds' imported but unused
> [Line 13:36](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L13): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 10
> [Line 20:26](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L20): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 10
> [Line 35:25](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L35): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 10
> [Line 51:5](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L51): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 9
> [Line 62:5](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L62): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 9
> [Line 84:5](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L84): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 10
> [Line 156:5](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L156): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 10
> [Line 186:25](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L186): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 10
> [Line 217:5](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L217): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 9
> [Line 269:5](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L269): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 9
> [Line 291:28](https://github.com/pydata/xarray/blob/f47af0519a77498a1c213ce0c5bd933408ecd866/xarray/tests/test_coarsen.py#L291): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 9







##### Comment last updated at 2021-06-16 16:40:59 UTC","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,922792799
https://github.com/pydata/xarray/pull/5474#issuecomment-862520948,https://api.github.com/repos/pydata/xarray/issues/5474,862520948,MDEyOklzc3VlQ29tbWVudDg2MjUyMDk0OA==,5635139,2021-06-16T16:22:25Z,2021-06-16T16:22:25Z,MEMBER,"Great!! We can import the fixtures, or we can promote them to `conftest.py` and pytest will automatically use them.

I had the same thing with flake8 IIRC; I think we can just `noqa` the imports.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,922792799