make aws_file_download return a path instead of empty list

getwilds · Oct 18, 2023 · a529636 · a529636
1 parent 8fc8659
commit a529636
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/R/files.R b/R/files.R
@@ -40,7 +40,7 @@ aws_file_upload <- function(bucket, path, key = basename(path), ...) {
 #' @inheritParams aws_file_upload
 #' @param ... named parameters passed on to [download_file](https://www.paws-r-sdk.com/docs/s3_download_file/)
 #' @details Wraps [download_file](https://www.paws-r-sdk.com/docs/s3_download_file/)
-#' @return `list` of length 0
+#' @return Path of downloaded file
 #' @examples \dontrun{
 #' temp_path <- tempfile()
 #' aws_file_download(bucket = "s64-test-2", key = "DESCRIPTION",
@@ -52,6 +52,7 @@ aws_file_upload <- function(bucket, path, key = basename(path), ...) {
 #' }
 aws_file_download <- function(bucket, key, path, ...) {
   env64$s3$download_file(Bucket = bucket, Key = key, Filename = path, ...)
+  return(path)
 }
 
 #' File attributes

diff --git a/man/aws_file_download.Rd b/man/aws_file_download.Rd