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/594#issuecomment-144540055,https://api.github.com/repos/pydata/xarray/issues/594,144540055,MDEyOklzc3VlQ29tbWVudDE0NDU0MDA1NQ==,1217238,2015-09-30T20:55:43Z,2015-09-30T20:55:43Z,MEMBER,"Sure, go ahead and set that up if you like. As long as it doesn't mark a build as failing, the info can't hurt. I'll note that I don't agree with many of it's ""errors"", though :). For example, as long as you write pure functions (which you should be doing anyways), it's perfectly fine to have mutable default argument.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,108983672
https://github.com/pydata/xarray/pull/594#issuecomment-144537098,https://api.github.com/repos/pydata/xarray/issues/594,144537098,MDEyOklzc3VlQ29tbWVudDE0NDUzNzA5OA==,2443309,2015-09-30T20:42:19Z,2015-09-30T20:42:19Z,MEMBER,"@shoyer - Thanks for merging.  How would you feel about adding [Lanscape](http://landscape.io) to our build checks?

If you haven't used Landscape before, its a style checker, much like what I was getting in my linter.  I ran it over my forks master branch to see what the output was like: https://landscape.io/github/jhamman/xray/2
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,108983672
https://github.com/pydata/xarray/pull/594#issuecomment-144485566,https://api.github.com/repos/pydata/xarray/issues/594,144485566,MDEyOklzc3VlQ29tbWVudDE0NDQ4NTU2Ng==,1217238,2015-09-30T17:36:20Z,2015-09-30T17:36:20Z,MEMBER,"OK, thanks!
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,108983672