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/991#issuecomment-244505226,https://api.github.com/repos/pydata/xarray/issues/991,244505226,MDEyOklzc3VlQ29tbWVudDI0NDUwNTIyNg==,1217238,2016-09-02T22:52:09Z,2016-09-02T22:52:09Z,MEMBER,"thanks!
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823
https://github.com/pydata/xarray/pull/991#issuecomment-244264366,https://api.github.com/repos/pydata/xarray/issues/991,244264366,MDEyOklzc3VlQ29tbWVudDI0NDI2NDM2Ng==,1217238,2016-09-02T02:03:20Z,2016-09-02T02:03:20Z,MEMBER,"This is almost there now.

I agree that we don't need to update the docs, but please do make note of it in ""What's New"".
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823
https://github.com/pydata/xarray/pull/991#issuecomment-243946499,https://api.github.com/repos/pydata/xarray/issues/991,243946499,MDEyOklzc3VlQ29tbWVudDI0Mzk0NjQ5OQ==,296686,2016-09-01T00:58:54Z,2016-09-01T00:58:54Z,CONTRIBUTOR,"The Travis builds seem to have all passed except for `py27-cdat+pynio` which has failed with a conda error that seems to be unrelated to these changes (`Error: HTTPError: 403 Client Error: Forbidden for url`)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823
https://github.com/pydata/xarray/pull/991#issuecomment-243933961,https://api.github.com/repos/pydata/xarray/issues/991,243933961,MDEyOklzc3VlQ29tbWVudDI0MzkzMzk2MQ==,296686,2016-08-31T23:32:56Z,2016-08-31T23:32:56Z,CONTRIBUTOR,"I think I've dealt with all of these.

Refactoring the tests has made them a bit cleaner and safer due to not using mutation - however I still have quite a few repeated lines of code. I've also combined them all into one test with lots of asserts - as otherwise I couldn't keep the helper functions within the test without repeating them loads of times.

Does anything need adding to the docs for this? I'm assuming not as it's just making the code do more checking, it's not adding a feature as such?
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823
https://github.com/pydata/xarray/pull/991#issuecomment-243007669,https://api.github.com/repos/pydata/xarray/issues/991,243007669,MDEyOklzc3VlQ29tbWVudDI0MzAwNzY2OQ==,1217238,2016-08-28T23:55:15Z,2016-08-28T23:55:15Z,MEMBER,"Take a look at the other netcdf tests -- they all use a decorator like
@requires_netCDF4, either on the class or the method. That ensures they are
skipped if required libraries for the functionality are not installed.
On Sun, Aug 28, 2016 at 2:43 PM Robin Wilson notifications@github.com
wrote:

> Thanks, I'll deal with these modifications.
> 
> Also, I'm seeing errors from TravisCI saying ValueError: cannot read or
> write netCDF files without netCDF4-python or scipy installed - what do I
> need to do to make sure these tests work on Travis? (They pass fine on my
> local machine)
> 
> —
> You are receiving this because you commented.
> Reply to this email directly, view it on GitHub
> https://github.com/pydata/xarray/pull/991#issuecomment-243001377, or mute
> the thread
> https://github.com/notifications/unsubscribe-auth/ABKS1qrd1mmelzaoL4omWV_TTLwAMeUSks5qkgEWgaJpZM4Ju5fU
> .
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823
https://github.com/pydata/xarray/pull/991#issuecomment-243001377,https://api.github.com/repos/pydata/xarray/issues/991,243001377,MDEyOklzc3VlQ29tbWVudDI0MzAwMTM3Nw==,296686,2016-08-28T21:43:49Z,2016-08-28T21:43:49Z,CONTRIBUTOR,"Thanks, I'll deal with these modifications.

Also, I'm seeing errors from TravisCI saying `ValueError: cannot read or write netCDF files without netCDF4-python or scipy installed` - what do I need to do to make sure these tests work on Travis? (They pass fine on my local machine)
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,173640823