chore: take advantage of existing context managers

Author: cpcloud

pins/boards.py

@@ -790,16 +790,13 @@ def _load_data(self, meta, pin_version_path):
 
     @contextlib.contextmanager
     def _open_pin_meta(self, path):
-        f = self.fs.open(path)
-        try:
+        with self.fs.open(path) as f:
             self._touch_cache(path)
 
             # optional additional data to put in Meta.local
             local = {}
 
             yield f, local
-        finally:
-            self.fs.close(f)
 
     def _get_cache_path(self, pin_name, version=None, fname=None):
         version_part = [version] if version is not None else []
@@ -1149,8 +1146,7 @@ def pin_versions_prune(self, *args, **kwargs):
 
     @contextlib.contextmanager
     def _open_pin_meta(self, path):
-        f = self.fs.open(path)
-        try:
+        with self.fs.open(path) as f:
             self._touch_cache(path)
 
             # optional additional data to put in Meta.local
@@ -1165,8 +1161,6 @@ def _open_pin_meta(self, path):
             }
 
             yield f, local
-        finally:
-            self.fs.close(f)
 
     def validate_pin_name(self, name) -> None:
         # this should be the default behavior, expecting a full pin name.