handle Raphael's suggested changes

pangeo-forge · Jan 22, 2024 · c2a83ae · c2a83ae
1 parent 742183f
commit c2a83ae
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/test-integration.yaml b/.github/workflows/test-integration.yaml
@@ -33,7 +33,7 @@ jobs:
         runner-version: [
           "pangeo-forge-runner==0.9.1",
           "pangeo-forge-runner==0.9.2",
-          "git+https://github.com/pangeo-forge/pangeo-forge-runner.git@gcorradini/add_back_context_manager#egg=pangeo-forge-runner",
+          "pangeo-forge-runner==0.9.3",
         ]
     steps:
       - uses: actions/checkout@v4

diff --git a/pangeo_forge_recipes/combiners.py b/pangeo_forge_recipes/combiners.py
@@ -65,7 +65,7 @@ class CombineMultiZarrToZarr(beam.CombineFn):
       along a dimension that does not exist in the individual inputs. In this latter
       case, precombining adds the additional dimension to the input so that its
       dimensionality will match that of the accumulator.
-    :param target_options: Storage options dict to pass to the MultiZarrToZarr
+    :param target_options: Target options dict to pass to the MultiZarrToZarr
 
     """
 

diff --git a/pangeo_forge_recipes/storage.py b/pangeo_forge_recipes/storage.py
@@ -119,7 +119,7 @@ def get_fsspec_remote_protocol(self):
         elif isinstance(fsspec_protocol, list):
             return fsspec_protocol[0]
         else:
-            raise ValueError("could not resolve fsspec protocol from underlying filesystem")
+            raise ValueError(f"could not resolve fsspec protocol '{fsspec_protocol}' from underlying filesystem")
 
     def get_mapper(self) -> fsspec.mapping.FSMap:
         """Get a mutable mapping object suitable for storing Zarr data."""