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/5174#issuecomment-830686289,https://api.github.com/repos/pydata/xarray/issues/5174,830686289,MDEyOklzc3VlQ29tbWVudDgzMDY4NjI4OQ==,5635139,2021-05-01T19:59:31Z,2021-05-01T19:59:31Z,MEMBER,Excellent — thank you @tammasloughran ! ,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-830433663,https://api.github.com/repos/pydata/xarray/issues/5174,830433663,MDEyOklzc3VlQ29tbWVudDgzMDQzMzY2Mw==,5635139,2021-04-30T22:27:23Z,2021-04-30T22:27:23Z,MEMBER,@tammasloughran this just needs a whatsnew now!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-830391695,https://api.github.com/repos/pydata/xarray/issues/5174,830391695,MDEyOklzc3VlQ29tbWVudDgzMDM5MTY5NQ==,14808389,2021-04-30T21:05:09Z,2021-04-30T21:05:09Z,MEMBER,"> A dask method `__dask_distributed_pack__` is complaining:

that seems to have been a conda issue (but it's gone now), and the remaining failure is a known flaky test, so CI looks good","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-830287979,https://api.github.com/repos/pydata/xarray/issues/5174,830287979,MDEyOklzc3VlQ29tbWVudDgzMDI4Nzk3OQ==,14371165,2021-04-30T18:40:58Z,2021-04-30T18:40:58Z,MEMBER,"The errors seems unrelated? Seems to rather be a mismatch in the CI? 

A dask method `__dask_distributed_pack__` is complaining:
```
>       from distributed.protocol.core import dumps_msgpack
E       ImportError: cannot import name 'dumps_msgpack' from 'distributed.protocol.core' (/usr/share/miniconda/envs/xarray-tests/lib
```
But those seems to have been removed in https://github.com/dask/distributed/pull/4677. Further reading in https://github.com/dask/dask-yarn/issues/147#issuecomment-829265029","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-830238152,https://api.github.com/repos/pydata/xarray/issues/5174,830238152,MDEyOklzc3VlQ29tbWVudDgzMDIzODE1Mg==,5635139,2021-04-30T17:13:07Z,2021-04-30T17:13:07Z,MEMBER,"@tammasloughran would be great to get this in by https://github.com/pydata/xarray/issues/5232 if you have any time, it's great work. I think it's fairly close to the tests passing.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-824241591,https://api.github.com/repos/pydata/xarray/issues/5174,824241591,MDEyOklzc3VlQ29tbWVudDgyNDI0MTU5MQ==,5635139,2021-04-21T17:43:51Z,2021-04-21T17:43:51Z,MEMBER,"After those two suggestions, let's merge?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-822712358,https://api.github.com/repos/pydata/xarray/issues/5174,822712358,MDEyOklzc3VlQ29tbWVudDgyMjcxMjM1OA==,2448579,2021-04-19T19:07:05Z,2021-04-19T19:07:05Z,MEMBER,"> I see now there is also a duplicate test here, which fails when using the newer dask version. I

That's testing the `Variable` object which is the lowest-level Xarray object. It would be a good place to add the assignment tests","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-822656799,https://api.github.com/repos/pydata/xarray/issues/5174,822656799,MDEyOklzc3VlQ29tbWVudDgyMjY1Njc5OQ==,2448579,2021-04-19T17:48:56Z,2021-04-19T17:48:56Z,MEMBER,Thanks @tammasloughran This is a great PR!,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-822460282,https://api.github.com/repos/pydata/xarray/issues/5174,822460282,MDEyOklzc3VlQ29tbWVudDgyMjQ2MDI4Mg==,5635139,2021-04-19T13:19:05Z,2021-04-19T13:19:05Z,MEMBER,"
> I don't quite understand understand what you mean. [Like this?](https://github.com/pydata/xarray/pull/5174/commits/6ddb76cf146907e0a9276c25393153ea5c7245a2)

Forgive me not being clear. The current test with the version check is great. And I hadn't realized there was a released dask version — even better that the test is already running on both versions of the code.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545
https://github.com/pydata/xarray/pull/5174#issuecomment-821563673,https://api.github.com/repos/pydata/xarray/issues/5174,821563673,MDEyOklzc3VlQ29tbWVudDgyMTU2MzY3Mw==,5635139,2021-04-16T21:04:52Z,2021-04-16T21:04:52Z,MEMBER,"Looks good! Not sure whether there's a reasonable way of testing this...

Are there any current tests for the error that break on the new version (i.e. because it doesn't raise an error)?","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,860073545