add suggestions from review

pydata · Nov 17, 2023 · e545137 · e545137
1 parent 66dffd5
commit e545137
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 3 deletions.
diff --git a/doc/whats-new.rst b/doc/whats-new.rst
@@ -40,8 +40,9 @@ Breaking changes
 - drop support for `cdms2 <https://github.com/CDAT/cdms>`_. Please use
   `xcdat <https://github.com/xCDAT/xcdat>`_ instead (:pull:`8441`).
   By `Justus Magin <https://github.com/keewis`_.
-
 - Bump minimum tested pint version to ``>=0.22``. By `Deepak Cherian <https://github.com/dcherian>`_.
+- Minimum supported versions for the following packages have changed: ``h5py >=3.7``, ``h5netcdf>=1.1``.
+  By `Kai Mühlbauer <https://github.com/kmuehlbauer>`_.
 
 Deprecations
 ~~~~~~~~~~~~

diff --git a/xarray/coding/variables.py b/xarray/coding/variables.py
@@ -564,7 +564,7 @@ def decode(self):
         raise NotImplementedError()
 
 
-class ObjectStringCoder(VariableCoder):
+class ObjectVLenStringCoder(VariableCoder):
     def encode(self):
         return NotImplementedError
 

diff --git a/xarray/conventions.py b/xarray/conventions.py
@@ -266,7 +266,7 @@ def decode_cf_variable(
         var = strings.EncodedStringCoder().decode(var)
 
     if original_dtype == object:
-        var = variables.ObjectStringCoder().decode(var)
+        var = variables.ObjectVLenStringCoder().decode(var)
         original_dtype = var.dtype
 
     if mask_and_scale: