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/2045#issuecomment-381742272,https://api.github.com/repos/pydata/xarray/issues/2045,381742272,MDEyOklzc3VlQ29tbWVudDM4MTc0MjI3Mg==,13837821,2018-04-16T20:42:01Z,2018-04-16T20:42:01Z,CONTRIBUTOR,"@shoyer Thank you for the helpful feedback. I think I addressed all of your comments, and also expanded the tests.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,312693611 https://github.com/pydata/xarray/pull/2045#issuecomment-380239938,https://api.github.com/repos/pydata/xarray/issues/2045,380239938,MDEyOklzc3VlQ29tbWVudDM4MDIzOTkzOA==,13837821,2018-04-10T20:42:34Z,2018-04-10T20:42:34Z,CONTRIBUTOR,"I only have a vague grasp of proper object oriented programming practices; not sure if the helper functions should be within the class or what, but it does seem to work this way.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,312693611 https://github.com/pydata/xarray/pull/2045#issuecomment-380156622,https://api.github.com/repos/pydata/xarray/issues/2045,380156622,MDEyOklzc3VlQ29tbWVudDM4MDE1NjYyMg==,13837821,2018-04-10T16:05:48Z,2018-04-10T16:05:48Z,CONTRIBUTOR,"OK, after some false starts, I think this is working, but the CI build is failing on CONDA_ENV=py35 and py36... but it's failing on a different test (specific to netcdf3, complaining about a temp file) and I can't see how it is impacted by my changes... thoughts?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,312693611