Compare ⋅ r-lib/pak

Overview Diff Coverage Changes Files Commits

No flags found

Use flags to group coverage reports by test type, project and/or folders.
Then setup custom commit statuses and notifications for each flag.

e.g., #unittest #integration

#production #enterprise

#frontend #backend

Learn more about Codecov Flags here.

Showing 1 of 2 files from the diff.

8.96% 0.00% -2.99%

R/system-requirements.R changed.

Other files ignored by Codecov

man/local_system_requirements.Rd has changed.

@@ -7,21 +7,27 @@

Loading

#' requirements for the queried operating system.
#'
#' @inheritParams local_install
#' @param os,os_release The operating system and operating system release version, see
#'   <https://github.com/rstudio/r-system-requirements#operating-systems> for the
#'   list of supported operating systems. If `NULL`, the default, these will be
#'   looked up using [distro::distro()].
#' @param os,os_release The operating system and operating system release
#'   version, e.g. "ubuntu", "debian", "centos", "redhat". See
#'   <https://github.com/rstudio/r-system-requirements#operating-systems> for
#'   all full list of supported operating systems.
#'
#'   If `NULL`, the default, these will be looked up using [distro::distro()].
#' @param execute,sudo If `execute` is `TRUE`, pak will execute the system
#'   commands (if any). If `sudo` is `TRUE`, pak will prepend the commands with
#'   [sudo](https://en.wikipedia.org/wiki/Sudo).
#' @param echo If `echo` is `TRUE` and `execute` is `TRUE`, echo the command output.
#' @return A character vector of commands needed to install the system requirements for the package (invisibly).
#' @return A character vector of commands needed to install the system
#'   requirements for the package.
#' @export
#' @examples
#' pkg_system_requirements("pak", "ubuntu", "20.04")
#' pkg_system_requirements("pak", "redhat", "7")
local_system_requirements <- function(os = NULL, os_release = NULL, root = ".", execute = FALSE, sudo = execute, echo = FALSE) {
  res <- remote(
    function(...) asNamespace("pak")$system_requirements_internal(...),
    list(os = os, os_release = os_release, root = root, package = NULL, execute = execute, sudo = sudo, echo = echo))
  invisible(res)
  if (execute) invisible(res) else res
}

#' @param package The package name to lookup system requirements for.

@@ -31,7 +37,7 @@

Loading

  res <- remote(
    function(...) asNamespace("pak")$system_requirements_internal(...),
    list(os = os, os_release = os_release, root = NULL, package = package, execute = execute, sudo = sudo, echo = echo))
  invisible(res)
  if (execute) invisible(res) else res
}

system_requirements_internal <- function(os, os_release, root, package, execute, sudo, echo) {

Everything is accounted for!

No changes detected that need to be reviewed.

What changes does Codecov check for?

Lines, not adjusted in diff, that have changed coverage data.

Files that introduced coverage data that had none before.

Files that have missing coverage data that once were tracked.

18.92% 0.00% -0.12%

System requirement tweaks

hadley

#248 6d9a371

CI Passed

-2

+2

Pull Request Base Commit

a1ccae8

Files		•	•	•	Coverage
R	ø	-2	ø	+2	^-0.12% 18.92%
Project Totals (23 files)	1,707	323	0	1,384	18.92%

Navigation	Overlay
`t` Navigate files	`h` Toggle hits
`y` Change url to tip of branch	`m` Toggle misses
`b / v` Jump to prev/next hit line	`p` Toggle partial
`z / x` Jump to prev/next missed or partial line	`1..9` Toggle flags
`shift + o` Open current page in GitHub	`a` Toggle all on
`/ or ?` Show keyboard shortcuts dialog	`c` Toggle context lines or commits

#248 System requirement tweaks

No flags found

Everything is accounted for!

What changes does Codecov check for?

1 Commits

7	7		#' requirements for the queried operating system.
8	8		#'
9	9		#' @inheritParams local_install
10		-	#' @param os,os_release The operating system and operating system release version, see
11		-	#' <https://github.com/rstudio/r-system-requirements#operating-systems> for the
12		-	#' list of supported operating systems. If `NULL`, the default, these will be
13		-	#' looked up using [distro::distro()].
	10	+	#' @param os,os_release The operating system and operating system release
	11	+	#' version, e.g. "ubuntu", "debian", "centos", "redhat". See
	12	+	#' <https://github.com/rstudio/r-system-requirements#operating-systems> for
	13	+	#' all full list of supported operating systems.
	14	+	#'
	15	+	#' If `NULL`, the default, these will be looked up using [distro::distro()].
14	16		#' @param execute,sudo If `execute` is `TRUE`, pak will execute the system
15	17		#' commands (if any). If `sudo` is `TRUE`, pak will prepend the commands with
16	18		#' [sudo](https://en.wikipedia.org/wiki/Sudo).
17	19		#' @param echo If `echo` is `TRUE` and `execute` is `TRUE`, echo the command output.
18		-	#' @return A character vector of commands needed to install the system requirements for the package (invisibly).
	20	+	#' @return A character vector of commands needed to install the system
	21	+	#' requirements for the package.
19	22		#' @export
	23	+	#' @examples
	24	+	#' pkg_system_requirements("pak", "ubuntu", "20.04")
	25	+	#' pkg_system_requirements("pak", "redhat", "7")
20	26		local_system_requirements <- function(os = NULL, os_release = NULL, root = ".", execute = FALSE, sudo = execute, echo = FALSE) {
21	27		res <- remote(
22	28		function(...) asNamespace("pak")$system_requirements_internal(...),
23	29		list(os = os, os_release = os_release, root = root, package = NULL, execute = execute, sudo = sudo, echo = echo))
24		-	invisible(res)
	30	+	if (execute) invisible(res) else res
25	31		}
26	32
27	33		#' @param package The package name to lookup system requirements for.