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/1919#issuecomment-385598298,https://api.github.com/repos/pydata/xarray/issues/1919,385598298,MDEyOklzc3VlQ29tbWVudDM4NTU5ODI5OA==,2443309,2018-05-01T04:44:52Z,2018-05-01T04:44:52Z,MEMBER,@fujiisoup - I think you can merge this whenever you're ready. ,"{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-377595079,https://api.github.com/repos/pydata/xarray/issues/1919,377595079,MDEyOklzc3VlQ29tbWVudDM3NzU5NTA3OQ==,5635139,2018-03-30T18:49:50Z,2018-03-30T18:49:50Z,MEMBER,"FWIW I think stickler is working v well, +1 to merge this","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-369117775,https://api.github.com/repos/pydata/xarray/issues/1919,369117775,MDEyOklzc3VlQ29tbWVudDM2OTExNzc3NQ==,12229877,2018-02-28T04:27:16Z,2018-02-28T04:27:16Z,CONTRIBUTOR,"Last time I used it Hound *didn't* update the PR status, so if so that removes one of my objections 😄  As a maintainer I prefer automating checks - I make mistakes, especially when tired, and it's nice knowing that the system has my back.

Hound only runs checks in the diff range though, so I think it's still valuable to keep running flake8 ourselves.  Trying both for a few months before making a final decision sounds like a good idea to me!","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-369112975,https://api.github.com/repos/pydata/xarray/issues/1919,369112975,MDEyOklzc3VlQ29tbWVudDM2OTExMjk3NQ==,5635139,2018-02-28T03:52:03Z,2018-02-28T03:52:03Z,MEMBER,"I share @fujiisoup 's perspective

But if people as wise as @Zac-HD disagree, let's leave Travis running for 1-2 months, see if it's helpful, and if it's not then we can turn it off. It's not immediately costly, and we'll gain understanding over the coming months.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-369112822,https://api.github.com/repos/pydata/xarray/issues/1919,369112822,MDEyOklzc3VlQ29tbWVudDM2OTExMjgyMg==,6815844,2018-02-28T03:50:57Z,2018-02-28T03:50:57Z,MEMBER,"Thanks for the information.
I do not have much experience about HoundCI but if it works well, it could be an alternative of stickler.

> they still check flake8 in Travis to ensure everything is fixed before merging.

But why is travis so special? 
HoundCI reports any flake8 violation not only as comments but also in *All checks have passed* part of PR, as travis does.
I think that maintainers can check any flake8 violation just from HoundCI, before merging.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-369111117,https://api.github.com/repos/pydata/xarray/issues/1919,369111117,MDEyOklzc3VlQ29tbWVudDM2OTExMTExNw==,12229877,2018-02-28T03:38:47Z,2018-02-28T03:39:07Z,CONTRIBUTOR,"[home-assistant](https://github.com/home-assistant/home-assistant) uses https://houndci.com, and it seems to work pretty well for them - it just leaves a review with any flake8 comments at the relevant lines of the pull.  

Note that while HoundCI is used to provide faster and more readable feedback to contributors, they still check flake8 in Travis to ensure everything is fixed before merging.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-369108264,https://api.github.com/repos/pydata/xarray/issues/1919,369108264,MDEyOklzc3VlQ29tbWVudDM2OTEwODI2NA==,6815844,2018-02-28T03:19:42Z,2018-02-28T03:19:42Z,MEMBER,"@Zac-HD, I don't mean to drop CI-based flake8 checking.
We are trying to move [stickler-ci](https://stickler-ci.com/), which is a CI specific for flake8 check.

I thought if stickler works fine, travis could concentrate on testing,
but stickler does not behave as what I have expected.
I agree that it is not a good idea to remove flake8 check from travis at this moment.

I think that in the early stage of the development it would be nicer if we could easily distinguish between testing failure and style inconsistency, but travis just says succeeded or not.
(it is possible to find it if we look deep inside the travis's log, but it requires some burden).

Do you have any idea to realize more explicit failure reporting?","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-369106191,https://api.github.com/repos/pydata/xarray/issues/1919,369106191,MDEyOklzc3VlQ29tbWVudDM2OTEwNjE5MQ==,12229877,2018-02-28T03:05:48Z,2018-02-28T03:05:48Z,CONTRIBUTOR,"I would be very disappointed to remove the flake8 check from Travis - in my experience, style guides are only applied *consistently* if they are enforced by CI.

Ensuring that each pull maintains our standards is much less burdensome than regular ""big push"" efforts to get back to the standard (see eg #1741 and #1824 - it took two people!), **and** avoids periods of lower code quality.

In short, I think that Travis *is* the right place to put these checks - review bots are a complement, not a substitute.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-366488172,https://api.github.com/repos/pydata/xarray/issues/1919,366488172,MDEyOklzc3VlQ29tbWVudDM2NjQ4ODE3Mg==,6815844,2018-02-18T02:42:27Z,2018-02-18T02:42:27Z,MEMBER,"OK. Let's wait a little more. 
It looks the rules are a little bit difference between flake8 in travis and stickler-ci.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911
https://github.com/pydata/xarray/pull/1919#issuecomment-366426281,https://api.github.com/repos/pydata/xarray/issues/1919,366426281,MDEyOklzc3VlQ29tbWVudDM2NjQyNjI4MQ==,10050469,2018-02-17T08:28:38Z,2018-02-17T08:28:38Z,MEMBER,"Yes, I would also wait a little more to see if we are happy with stickler","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,297794911