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/2482#issuecomment-434134473,https://api.github.com/repos/pydata/xarray/issues/2482,434134473,MDEyOklzc3VlQ29tbWVudDQzNDEzNDQ3Mw==,1217238,2018-10-30T01:01:17Z,2018-10-30T01:01:17Z,MEMBER,Thanks @TomNicholas !,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,369673042
https://github.com/pydata/xarray/pull/2482#issuecomment-429427849,https://api.github.com/repos/pydata/xarray/issues/2482,429427849,MDEyOklzc3VlQ29tbWVudDQyOTQyNzg0OQ==,24736507,2018-10-12T19:01:20Z,2018-10-29T15:19:05Z,NONE,"Hello @TomNicholas! Thanks for updating the PR.

 - There are no PEP8 issues in the file [`xarray/core/common.py`](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/core/common.py) !

 - There are no PEP8 issues in the file [`xarray/core/dataarray.py`](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/core/dataarray.py) !

 - There are no PEP8 issues in the file [`xarray/core/dataset.py`](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/core/dataset.py) !

 - There are no PEP8 issues in the file [`xarray/core/groupby.py`](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/core/groupby.py) !

 - In the file [`xarray/core/options.py`](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/core/options.py), following                are the PEP8 issues :
> [Line 56:80](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/core/options.py#L56): [E501](https://duckduckgo.com/?q=pep8%20E501) line too long (97 > 79 characters)


 - There are no PEP8 issues in the file [`xarray/core/resample.py`](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/core/resample.py) !

 - There are no PEP8 issues in the file [`xarray/core/variable.py`](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/core/variable.py) !

 - There are no PEP8 issues in the file [`xarray/tests/test_options.py`](https://github.com/TomNicholas/xarray/blob/cbf227d4bef2cb270e4ec9cfcee7024625041cec/xarray/tests/test_options.py) !



##### Comment last updated on October 29, 2018 at 15:19 Hours UTC","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,369673042
https://github.com/pydata/xarray/pull/2482#issuecomment-433628471,https://api.github.com/repos/pydata/xarray/issues/2482,433628471,MDEyOklzc3VlQ29tbWVudDQzMzYyODQ3MQ==,6628425,2018-10-27T15:06:35Z,2018-10-27T15:06:35Z,MEMBER,"@TomNicholas thanks, things look better now.  Don't worry about the Appveyor failures; a new version of `cftime` (installed via `pip` on the Python 2 builds there) was just released and there is a bug they are still ironing out on a few platforms (including Windows): https://github.com/conda-forge/cftime-feedstock/pull/10, https://github.com/Unidata/cftime/pull/76.  ","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,369673042
https://github.com/pydata/xarray/pull/2482#issuecomment-433623298,https://api.github.com/repos/pydata/xarray/issues/2482,433623298,MDEyOklzc3VlQ29tbWVudDQzMzYyMzI5OA==,6628425,2018-10-27T14:05:53Z,2018-10-27T14:05:53Z,MEMBER,@TomNicholas it looks like the merge conflict in `whats-new.rst` is preventing the CI from running.  I think if you sync with the latest master branch and resolve the conflict things should work properly.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,369673042
https://github.com/pydata/xarray/pull/2482#issuecomment-433621969,https://api.github.com/repos/pydata/xarray/issues/2482,433621969,MDEyOklzc3VlQ29tbWVudDQzMzYyMTk2OQ==,35968931,2018-10-27T13:49:53Z,2018-10-27T13:49:53Z,MEMBER,"No idea why the CI test fails now - I literally changed one line, from `class TestAttrRetention:` to
`class TestAttrRetention(object):`, and all tests pass on my local machine.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,369673042
https://github.com/pydata/xarray/pull/2482#issuecomment-431099046,https://api.github.com/repos/pydata/xarray/issues/2482,431099046,MDEyOklzc3VlQ29tbWVudDQzMTA5OTA0Ng==,1217238,2018-10-18T17:41:43Z,2018-10-18T17:41:43Z,MEMBER,"@TomNicholas I'm not sure, but don't worry about cleaning up git history with rebase. We squash all commits before merging anyways, and GitHub's review tools work a little better when you don't rebase.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,369673042
https://github.com/pydata/xarray/pull/2482#issuecomment-431098280,https://api.github.com/repos/pydata/xarray/issues/2482,431098280,MDEyOklzc3VlQ29tbWVudDQzMTA5ODI4MA==,35968931,2018-10-18T17:39:24Z,2018-10-18T17:39:24Z,MEMBER,"I wanted to clean up my commits using git rebase, but when I tried `git rebase -i` I only had the choice to alter my last 5 commits to this branch, not all 8. I'm not sure if you know what I'm doing wrong there?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,369673042
https://github.com/pydata/xarray/pull/2482#issuecomment-431096301,https://api.github.com/repos/pydata/xarray/issues/2482,431096301,MDEyOklzc3VlQ29tbWVudDQzMTA5NjMwMQ==,1217238,2018-10-18T17:33:20Z,2018-10-18T17:33:20Z,MEMBER,Unit tests and docs look good.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,369673042