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/153#issuecomment-45960686,https://api.github.com/repos/pydata/xarray/issues/153,45960686,MDEyOklzc3VlQ29tbWVudDQ1OTYwNjg2,1217238,2014-06-12T23:31:51Z,2014-06-12T23:31:51Z,MEMBER,"OK, I guess this is fine -- I merged the other PR.

I will open another issue for exposing a public interface to decoding functions.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,35564268
https://github.com/pydata/xarray/pull/153#issuecomment-45949753,https://api.github.com/repos/pydata/xarray/issues/153,45949753,MDEyOklzc3VlQ29tbWVudDQ1OTQ5NzUz,1217238,2014-06-12T21:22:26Z,2014-06-12T21:22:26Z,MEMBER,"No, just that it's unclear if it's a good idea to add tests (or
functionality) to internal functions in ways for features that aren't used
internally.

On Thu, Jun 12, 2014 at 1:29 PM, akleeman notifications@github.com wrote:

> I'm confused, are you saying we shouldn't add tests for internal functions?
> 
> —
> Reply to this email directly or view it on GitHub
> https://github.com/xray/xray/pull/153#issuecomment-45943780.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,35564268
https://github.com/pydata/xarray/pull/153#issuecomment-45921269,https://api.github.com/repos/pydata/xarray/issues/153,45921269,MDEyOklzc3VlQ29tbWVudDQ1OTIxMjY5,1217238,2014-06-12T17:19:23Z,2014-06-12T17:19:23Z,MEMBER,"I'm not opposed to this in principle, but this is an internal API that wasn't really for external consumption.

If you're relying on this in external code, then we should think about what the public API should be and add tests there, not for this internal function.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,35564268