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/1393#issuecomment-302955973,https://api.github.com/repos/pydata/xarray/issues/1393,302955973,MDEyOklzc3VlQ29tbWVudDMwMjk1NTk3Mw==,1217238,2017-05-21T19:04:26Z,2017-05-21T19:04:26Z,MEMBER,thanks!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225672435
https://github.com/pydata/xarray/pull/1393#issuecomment-298966173,https://api.github.com/repos/pydata/xarray/issues/1393,298966173,MDEyOklzc3VlQ29tbWVudDI5ODk2NjE3Mw==,1217238,2017-05-03T16:38:02Z,2017-05-03T16:38:02Z,MEMBER,"I'd be slightly more comfortable if we had this being unit tested on Travis-CI, but that's somewhat tricky to setup because we don't wan to require network access for the test suite by default. You could do this by adding a flag like `--run-network-tests` in `conftest.py`, and then hooking it through in `xarray/tests/__init__.py` like `--run-flaky`. I would be awesome if you could take a look at that and set the Travis/Appveyor tests to test the tutorial, but isn't strictly required to merge this.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225672435
https://github.com/pydata/xarray/pull/1393#issuecomment-298964022,https://api.github.com/repos/pydata/xarray/issues/1393,298964022,MDEyOklzc3VlQ29tbWVudDI5ODk2NDAyMg==,1217238,2017-05-03T16:30:01Z,2017-05-03T16:30:01Z,MEMBER,"Thanks @gidden, this look great to me. Could you kindly add a brief note to the ""Bug fixes"" section of `whats-new.rst`?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225672435
https://github.com/pydata/xarray/pull/1393#issuecomment-298737305,https://api.github.com/repos/pydata/xarray/issues/1393,298737305,MDEyOklzc3VlQ29tbWVudDI5ODczNzMwNQ==,1217238,2017-05-02T19:32:16Z,2017-05-02T19:32:16Z,MEMBER,"Thanks for taking this on @gidden.

> Also not sure how best to import shutil. I followed the current pattern for os, but am not sure why that pattern exists in this file.

I think this exists to work around the fact that IPython shows everything that isn't private in auto-complete (https://github.com/ipython/ipykernel/issues/129). But we should probably switch back to normal imports in this module -- this is really more of an IPython issue.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,225672435