revert docstring

lincbrain · Jul 18, 2024 · 81639be · 81639be
1 parent 78edc36
commit 81639be
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 7 deletions.
diff --git a/lincbrain/dandiapi.py b/lincbrain/dandiapi.py
@@ -644,7 +644,7 @@ def check_schema_version(self, schema_version: str | None = None) -> None:
         """
         if schema_version is None:
             schema_version = models.get_schema_version()
-        server_info = self.get("/info/")  # 0.6.7
+        server_info = self.get("/info/")
         server_schema_version = server_info.get("schema_version")
         if not server_schema_version:
             raise RuntimeError(

diff --git a/lincbrain/move.py b/lincbrain/move.py
@@ -801,10 +801,6 @@ def move(
     if dandiset is None:
         dandiset = Path()
     with ExitStack() as stack:
-        client = DandiAPIClient.for_dandi_instance(dandi_instance)
-        client.dandi_authenticate()
-        stack.enter_context(client)
-
         mover: Mover
         client: DandiAPIClient | None = None
         if work_on is MoveWorkOn.AUTO:
@@ -815,6 +811,8 @@ def move(
             if isinstance(dandiset, str):
                 raise TypeError("`dandiset` must be a Path when work_on='both'")
             local_ds, subpath = find_dandiset_and_subpath(dandiset)
+            client = DandiAPIClient.for_dandi_instance(dandi_instance)
+            stack.enter_context(client)
             remote_ds = client.get_dandiset(
                 local_ds.identifier, version_id="draft", lazy=False
             )
@@ -835,14 +833,15 @@ def move(
                 if not isinstance(url, DandisetURL):
                     raise ValueError("URL does not point to a Dandiset")
                 client = url.get_client()
-                client.dandi_authenticate()
                 stack.enter_context(client)
                 rds = url.get_dandiset(client, lazy=False)
                 assert rds is not None
                 remote_ds = rds
                 subpath = Path()
             else:
                 local_ds, subpath = find_dandiset_and_subpath(dandiset)
+                client = DandiAPIClient.for_dandi_instance(dandi_instance)
+                stack.enter_context(client)
                 remote_ds = client.get_dandiset(
                     local_ds.identifier, version_id="draft", lazy=False
                 )

diff --git a/lincbrain/upload.py b/lincbrain/upload.py
@@ -94,7 +94,7 @@ def upload(
         # session gets properly closed.  Otherwise, pytest sometimes complains
         # under obscure conditions.
         client = stack.enter_context(DandiAPIClient.for_dandi_instance(dandi_instance))
-        client.check_schema_version()  # Aaron
+        client.check_schema_version()
         client.dandi_authenticate()
 
         if os.environ.get("DANDI_DEVEL_INSTRUMENT_REQUESTS_SUPERLEN"):