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/3804#issuecomment-649053064,https://api.github.com/repos/pydata/xarray/issues/3804,649053064,MDEyOklzc3VlQ29tbWVudDY0OTA1MzA2NA==,1197350,2020-06-24T20:27:56Z,2020-06-24T20:27:56Z,MEMBER,"Sorry for letting this hang so long. Now that https://github.com/zarr-developers/zarr-python/pull/557 has been merged and zarr 2.4.0 has been released, we should be able to finish this off.

The easiest thing would be to just require zarr > 2.4.0. However, we currently don't require zarr at all...it's an optional dependency. The code in this PR will fail for older versions of zarr. What's the best way to handle this? Having a bunch of internal checks for zarr versions feels clunky, but we've done it before.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,572251686
https://github.com/pydata/xarray/pull/3804#issuecomment-592097638,https://api.github.com/repos/pydata/xarray/issues/3804,592097638,MDEyOklzc3VlQ29tbWVudDU5MjA5NzYzOA==,1197350,2020-02-27T18:05:35Z,2020-03-02T14:40:30Z,MEMBER,"Fantastic, thanks @ajelenak so much for this contribution!

Two items come to mind:
- [x] this will definitely need a test, just something simple to verify that the `chunk_store` argument works as expected
- [ ] curious what will happen if you pass an actual Zarr [storage](https://zarr.readthedocs.io/en/stable/api/storage.html#module-zarr.storage) object. I assume it will work fine, since these are all mutable mappings.

","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,572251686