Skip to content

Commit

Permalink
Documented args not in usage per CRAN check
Browse files Browse the repository at this point in the history
  • Loading branch information
daranzolin committed Apr 23, 2024
1 parent 01da6b5 commit 8959bff
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 8 deletions.
7 changes: 3 additions & 4 deletions R/tar_sql_raw.R
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ tar_sql_raw <- function(
cue = cue
)

# downstream target
# Downstream target
deps <- sort(unique(sql_deps(path)))
target_output <- targets::tar_target_raw(
name = name,
Expand Down Expand Up @@ -161,7 +161,6 @@ tar_sql_command <- function(
#' @export
#' @keywords internal
#' @return a data frame.
#' @param path Path to the SQL query.
#' @param args A named list of arguments to `glue::glue_sql()`.
#' @param deps An unnamed list of target dependencies of the R Markdown
#' report, automatically created by `tar_sql_deps()`.
Expand All @@ -177,8 +176,8 @@ tar_sql_exec <- function(args, deps) {
#' Users should not invoke it directly.
#' @export
#' @keywords internal
#' @return a relative file path.
#' @param path Path to the SQL query.
#' @return A relative file path.
#' @param args Passed to `fs::path_rel()`.
tar_sql_file <- function(args) {
gc()
do.call(what = fs::path_rel, args)
Expand Down
2 changes: 0 additions & 2 deletions man/tar_sql_exec.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions man/tar_sql_file.Rd

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 8959bff

Please sign in to comment.