Skip to content

Commit

Permalink
simplify capturing R dependencies message (#1099)
Browse files Browse the repository at this point in the history
don't mention renv as this is confusing

Signed-off-by: Elliot Murphy <[email protected]>
  • Loading branch information
statik committed Aug 30, 2024
1 parent 9999fdb commit 2fe8fde
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion R/bundlePackage.R
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ computePackageDependencies <- function(bundleDir,
# from the bundle (retaining the renv.lock).
removeRenv(bundleDir, lockfile = FALSE)
} else {
taskStart(quiet, "Capturing R dependencies with renv")
taskStart(quiet, "Capturing R dependencies")
# TODO: give user option to choose between implicit and explicit
deps <- snapshotRenvDependencies(bundleDir, extraPackages, quiet = quiet, verbose = verbose)
}
Expand Down
4 changes: 2 additions & 2 deletions tests/testthat/_snaps/bundlePackage.md
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
Code
pkgs <- bundlePackages(app_dir)
Message
i Capturing R dependencies with renv
i Capturing R dependencies
v Found 2 dependencies

# can snapshot deps with packrat (option)
Expand Down Expand Up @@ -43,7 +43,7 @@
Code
. <- bundlePackages(app_dir)
Message
i Capturing R dependencies with renv
i Capturing R dependencies
v Found 1 dependency
Condition
Error:
Expand Down

0 comments on commit 2fe8fde

Please sign in to comment.