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/5425#issuecomment-854940045,https://api.github.com/repos/pydata/xarray/issues/5425,854940045,MDEyOklzc3VlQ29tbWVudDg1NDk0MDA0NQ==,37740986,2021-06-04T19:00:58Z,2021-06-04T19:00:58Z,CONTRIBUTOR,"Yes I understand now
I committed myself to do the last minor adjustment on github, I really should not have *facepalm* (pre-commit is great)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,908651787
https://github.com/pydata/xarray/pull/5425#issuecomment-854127828,https://api.github.com/repos/pydata/xarray/issues/5425,854127828,MDEyOklzc3VlQ29tbWVudDg1NDEyNzgyOA==,37740986,2021-06-03T19:39:42Z,2021-06-03T19:49:01Z,CONTRIBUTOR,"> Ha, no problem. And don't worry about the commit quality — it's all squashed into one at the end. I generally write one good message at the start but that's all.

Good that they are going to be squashed, because this PR starts to look very much chaotic ah ah  
So when they are squashed, the message of the first commit get automatically picked ?

> One small change — could we say in the error message that they all need to be either all `DataArray`s or all `Dataset`s? For a new user, the currently proposed messages could suggest that they all need to be the type in the message, and that type is determined by the order of the args, which is a bit confusing. That might mean the error message is the same for both, or only slightly different. Does that make sense?

It does make sense  
Done ✔

I also edited whats new
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,908651787
https://github.com/pydata/xarray/pull/5425#issuecomment-853634406,https://api.github.com/repos/pydata/xarray/issues/5425,853634406,MDEyOklzc3VlQ29tbWVudDg1MzYzNDQwNg==,37740986,2021-06-03T07:13:57Z,2021-06-03T07:20:05Z,CONTRIBUTOR,"Oh, I just found, I wanted to do three separate commits for each changes, but when I did the dataarray [commit](https://github.com/pydata/xarray/pull/5425/commits/462d72045a6a3407b2beee524e420778c88c4f78), I somehow reverted the changes of the dataset commit *facepalm*. All the tests were working on my computer because I still had the changes locally. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,908651787
https://github.com/pydata/xarray/pull/5425#issuecomment-853345364,https://api.github.com/repos/pydata/xarray/issues/5425,853345364,MDEyOklzc3VlQ29tbWVudDg1MzM0NTM2NA==,37740986,2021-06-02T20:01:43Z,2021-06-02T20:02:24Z,CONTRIBUTOR,"Ok I'll do the whatsnew soon :)
I am a little bit stuck though, I don't really found why my test would cause a ValueError in those tests","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,908651787
https://github.com/pydata/xarray/pull/5425#issuecomment-852441855,https://api.github.com/repos/pydata/xarray/issues/5425,852441855,MDEyOklzc3VlQ29tbWVudDg1MjQ0MTg1NQ==,37740986,2021-06-01T21:00:47Z,2021-06-01T21:00:47Z,CONTRIBUTOR,"> 
> 
> This looks great! Do you want to add a note to the whatsnew?

What is the whatsnew ? 
I am still checking why there is some test failing 😶","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,908651787