Merge pull request #688 from christian-monch/nail_down_kwargs

consolidate keyword argument indicator

datalad_next/patches/replace_sshremoteio.py

@@ -207,7 +207,7 @@ def put(self, src, dst, progress_cb):
             PurePosixPath(dst),
             # the given callback only takes a single int, but posix.upload
             # gives two (cur, target) -> have an adaptor
-            lambda c, m: progress_cb(c),
+            progress_callback=lambda c, m: progress_cb(c),
             check=True,
         )
 
@@ -218,7 +218,7 @@ def get(self, src, dst, progress_cb):
             Path(dst),
             # the given callback only takes a single int, but posix.download
             # gives two (cur, target) -> have an adaptor
-            lambda c, m: progress_cb(c),
+            progress_callback=lambda c, m: progress_cb(c),
             check=True,
         )
 

datalad_next/shell/operations/posix.py

@@ -51,6 +51,7 @@ def upload(
     shell: ShellCommandExecutor,
     local_path: Path,
     remote_path: PurePosixPath,
+    *,
     progress_callback: Callable[[int, int], None] | None = None,
     check: bool = False,
 ) -> ExecutionResult:
@@ -96,7 +97,11 @@ def upload(
     """
 
     def signaling_read(
-        file: BinaryIO, size: int, queue: Queue, chunk_size: int = COPY_BUFSIZE
+            file: BinaryIO,
+            size: int,
+            queue: Queue,
+            *,
+            chunk_size: int = COPY_BUFSIZE
     ):
         """iterator that reads from a file and signals EOF via a queue
 
@@ -158,8 +163,8 @@ def download(
     shell: ShellCommandExecutor,
     remote_path: PurePosixPath,
     local_path: Path,
-    progress_callback: Callable[[int, int], None] | None = None,
     *,
+    progress_callback: Callable[[int, int], None] | None = None,
     response_generator_class: type[
         DownloadResponseGenerator
     ] = DownloadResponseGeneratorPosix,