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/5476#issuecomment-867818463,https://api.github.com/repos/pydata/xarray/issues/5476,867818463,MDEyOklzc3VlQ29tbWVudDg2NzgxODQ2Mw==,41898282,2021-06-24T17:24:29Z,2021-06-24T17:24:29Z,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 8a338eeb. ± Comparison against base commit 8a338eeb. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,922843005 https://github.com/pydata/xarray/pull/5476#issuecomment-867800319,https://api.github.com/repos/pydata/xarray/issues/5476,867800319,MDEyOklzc3VlQ29tbWVudDg2NzgwMDMxOQ==,2448579,2021-06-24T16:55:21Z,2021-06-24T16:55:21Z,MEMBER,Thanks for the review @mathause ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,922843005 https://github.com/pydata/xarray/pull/5476#issuecomment-862535687,https://api.github.com/repos/pydata/xarray/issues/5476,862535687,MDEyOklzc3VlQ29tbWVudDg2MjUzNTY4Nw==,24736507,2021-06-16T16:42:26Z,2021-06-23T16:18:22Z,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/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py): > [Line 16:1](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L16): [F401](https://duckduckgo.com/?q=pep8%20F401) '.test_dataarray.da' imported but unused > [Line 17:1](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L17): [F401](https://duckduckgo.com/?q=pep8%20F401) '.test_dataset.ds' imported but unused > [Line 20:36](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L20): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 17 > [Line 27:26](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L27): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 17 > [Line 42:25](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L42): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 17 > [Line 58:5](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L58): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 16 > [Line 69:5](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L69): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 16 > [Line 91:5](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L91): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 17 > [Line 163:5](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L163): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 17 > [Line 193:25](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L193): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 17 > [Line 224:5](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L224): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 16 > [Line 276:5](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L276): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 16 > [Line 298:28](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L298): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'da' from line 16 > [Line 314:5](https://github.com/pydata/xarray/blob/56d8c368591ed9a177f934095b284aa1a5d6ac9f/xarray/tests/test_coarsen.py#L314): [F811](https://duckduckgo.com/?q=pep8%20F811) redefinition of unused 'ds' from line 17 ##### Comment last updated at 2021-06-23 16:18:22 UTC","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,922843005 https://github.com/pydata/xarray/pull/5476#issuecomment-865191078,https://api.github.com/repos/pydata/xarray/issues/5476,865191078,MDEyOklzc3VlQ29tbWVudDg2NTE5MTA3OA==,2448579,2021-06-21T16:52:00Z,2021-06-21T16:52:00Z,MEMBER,This should be ready for review & merge,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,922843005