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/4431#issuecomment-697933542,https://api.github.com/repos/pydata/xarray/issues/4431,697933542,MDEyOklzc3VlQ29tbWVudDY5NzkzMzU0Mg==,1217238,2020-09-23T19:44:46Z,2020-09-23T19:44:46Z,MEMBER,"> @jhamman I don't understand why [4f414f2](https://github.com/pydata/xarray/commit/4f414f2d5eb2e5a12fb8ae1012c5ac7aa43b6f0b) is shown here, it is a commit in master.

I usually find it easiest just to use ""merge master"" to integrate the latest changes, and to treat pull requests as ""append only"". History gets consolidated/rewritten when we merge the PR with ""Squash and Merge""","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109
https://github.com/pydata/xarray/pull/4431#issuecomment-697924785,https://api.github.com/repos/pydata/xarray/issues/4431,697924785,MDEyOklzc3VlQ29tbWVudDY5NzkyNDc4NQ==,226037,2020-09-23T19:26:44Z,2020-09-23T19:26:44Z,MEMBER,"@jhamman I don't understand why 4f414f2 is shown here, it is a commit in master.

As for the rest, we re applied the engine selection refactor and added the new `zarr` bits.

All looks reasonable to me and all tests pass.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109
https://github.com/pydata/xarray/pull/4431#issuecomment-696970366,https://api.github.com/repos/pydata/xarray/issues/4431,696970366,MDEyOklzc3VlQ29tbWVudDY5Njk3MDM2Ng==,226037,2020-09-22T20:45:53Z,2020-09-22T20:45:53Z,MEMBER,"Thanks @keewis! So it is correct, I wanted to reset to master and re apply the changes.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109
https://github.com/pydata/xarray/pull/4431#issuecomment-696963578,https://api.github.com/repos/pydata/xarray/issues/4431,696963578,MDEyOklzc3VlQ29tbWVudDY5Njk2MzU3OA==,14808389,2020-09-22T20:32:37Z,2020-09-22T20:32:37Z,MEMBER,"the reason the PR was closed is that you removed the changes, so the diff to `master` is empty. I think you can reopen after pushing commits with changes.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109
https://github.com/pydata/xarray/pull/4431#issuecomment-696963505,https://api.github.com/repos/pydata/xarray/issues/4431,696963505,MDEyOklzc3VlQ29tbWVudDY5Njk2MzUwNQ==,221526,2020-09-22T20:32:28Z,2020-09-22T20:32:28Z,CONTRIBUTOR,"@alexamici Force-pushing doesn't normally close it, so this is weird...","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109
https://github.com/pydata/xarray/pull/4431#issuecomment-696960812,https://api.github.com/repos/pydata/xarray/issues/4431,696960812,MDEyOklzc3VlQ29tbWVudDY5Njk2MDgxMg==,226037,2020-09-22T20:27:02Z,2020-09-22T20:27:02Z,MEMBER,"Woops I didn't know that force pushing a branch would close the PR :/

I'll try to reopen this PR or create a new one. Sorry for the noise.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109
https://github.com/pydata/xarray/pull/4431#issuecomment-696518718,https://api.github.com/repos/pydata/xarray/issues/4431,696518718,MDEyOklzc3VlQ29tbWVudDY5NjUxODcxOA==,1217238,2020-09-22T05:41:44Z,2020-09-22T05:41:44Z,MEMBER,"https://github.com/pydata/xarray/pull/4187 has been merged, so this can go ahead again!","{""total_count"": 2, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 2, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109
https://github.com/pydata/xarray/pull/4431#issuecomment-694539990,https://api.github.com/repos/pydata/xarray/issues/4431,694539990,MDEyOklzc3VlQ29tbWVudDY5NDUzOTk5MA==,2443309,2020-09-17T22:46:47Z,2020-09-17T22:46:47Z,MEMBER,Thanks for the ping @alexamici - this looks good to me. Happy to see this merged once we sort out the conflicts in #4187.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109
https://github.com/pydata/xarray/pull/4431#issuecomment-694411881,https://api.github.com/repos/pydata/xarray/issues/4431,694411881,MDEyOklzc3VlQ29tbWVudDY5NDQxMTg4MQ==,226037,2020-09-17T18:16:53Z,2020-09-17T18:17:14Z,MEMBER,"@jhamman: @shoyer  requested to keep this PR on hold until #4187 is merged as it will produce conflicts.

So I converted the PR to draft, after #4187 gets in, we will fix the conflicts and resubmit.","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,703550109