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/issues/1954#issuecomment-370044335,https://api.github.com/repos/pydata/xarray/issues/1954,370044335,MDEyOklzc3VlQ29tbWVudDM3MDA0NDMzNQ==,1197350,2018-03-02T20:32:59Z,2018-03-02T20:32:59Z,MEMBER,"I think you're right. Since zarr 2.2 is being released so soon, it's probably silly to add such a check.
","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370043488,https://api.github.com/repos/pydata/xarray/issues/1954,370043488,MDEyOklzc3VlQ29tbWVudDM3MDA0MzQ4OA==,2443309,2018-03-02T20:29:49Z,2018-03-02T20:29:49Z,MEMBER,"Okay, this is just a little messy right now because we're ahead of zarr in their release schedule. We expect that will come out on Monday (see https://github.com/zarr-developers/zarr/issues/242). 

Typically, we have stayed away from checking versions of all optional dependencies on import this may be one time that would be useful. The problem is these checks often fall out of date end up causing unnecessary clutter down the line. 

To resolve this we can add a version check (as you've shown above) or we can wait until Monday. I'm also not opposed to mentioning in the documentation what minimum version we support is. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370042719,https://api.github.com/repos/pydata/xarray/issues/1954,370042719,MDEyOklzc3VlQ29tbWVudDM3MDA0MjcxOQ==,1197350,2018-03-02T20:26:24Z,2018-03-02T20:26:48Z,MEMBER,"@kaipak it would be great if you could put together a quick PR that would raise an error on importing zarr if the version is too old.

You would insert this around here:
https://github.com/pydata/xarray/blob/master/xarray/backends/zarr.py#L273 

The code would looks something like:
```python
from distutils.version import LooseVersion
...
import zarr
if LooseVersion(zarr.__version__) < '2.2':
    raise NotImplementedError(""Zarr version 2.2 or greater is required by xarray.  ""
       ""See zarr installation instructions at http://zarr.readthedocs.io/en/stable/#installation"")","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370041032,https://api.github.com/repos/pydata/xarray/issues/1954,370041032,MDEyOklzc3VlQ29tbWVudDM3MDA0MTAzMg==,1197350,2018-03-02T20:19:40Z,2018-03-02T20:19:40Z,MEMBER,I guess we should be raising an error if zarr is too old rather than failing in this way.,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370040272,https://api.github.com/repos/pydata/xarray/issues/1954,370040272,MDEyOklzc3VlQ29tbWVudDM3MDA0MDI3Mg==,1217238,2018-03-02T20:16:32Z,2018-03-02T20:16:32Z,MEMBER,"We should document this more clearly, but you need zarr 2.2 (currently in release candidate stage) for use with xarray.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608
https://github.com/pydata/xarray/issues/1954#issuecomment-370037927,https://api.github.com/repos/pydata/xarray/issues/1954,370037927,MDEyOklzc3VlQ29tbWVudDM3MDAzNzkyNw==,1197350,2018-03-02T20:06:38Z,2018-03-02T20:06:38Z,MEMBER,Can you show the output of ``xr.show_versions()``?,"{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,301888608