clean up internal namespace and directly re-export nflreadr functions

DESCRIPTION

@@ -52,6 +52,7 @@ Imports:
     glue,
     janitor,
     lifecycle,
+    magrittr,
     mgcv,
     nflreadr (>= 1.2.0),
     progressr (>= 0.6.0),

NAMESPACE

@@ -19,50 +19,28 @@ export(fast_scraper_roster)
 export(fast_scraper_schedules)
 export(load_pbp)
 export(load_player_stats)
+export(load_rosters)
+export(load_schedules)
 export(missing_raw_pbp)
 export(nflverse_sitrep)
-export(report)
 export(save_raw_pbp)
 export(update_db)
-import(dplyr)
 import(fastrmodels)
-importFrom(cli,rule)
-importFrom(curl,curl_fetch_memory)
 importFrom(data.table,"%between%")
 importFrom(data.table,"%chin%")
 importFrom(data.table,setDT)
-importFrom(furrr,future_map)
-importFrom(furrr,future_map_chr)
-importFrom(furrr,future_map_dfr)
-importFrom(future,plan)
-importFrom(glue,glue)
-importFrom(glue,glue_sql)
-importFrom(janitor,clean_names)
-importFrom(lifecycle,deprecated)
-importFrom(mgcv,predict.bam)
+importFrom(magrittr,"%>%")
+importFrom(nflreadr,load_pbp)
+importFrom(nflreadr,load_player_stats)
+importFrom(nflreadr,load_rosters)
+importFrom(nflreadr,load_schedules)
 importFrom(nflreadr,nflverse_sitrep)
-importFrom(progressr,progressor)
+importFrom(rlang,"%||%")
 importFrom(rlang,":=")
 importFrom(rlang,.data)
 importFrom(rlang,.env)
 importFrom(rlang,dots_list)
 importFrom(rlang,inform)
 importFrom(stats,na.omit)
 importFrom(stats,predict)
-importFrom(stringr,str_detect)
-importFrom(stringr,str_extract)
-importFrom(stringr,str_extract_all)
-importFrom(stringr,str_length)
-importFrom(stringr,str_remove_all)
-importFrom(stringr,str_replace_all)
-importFrom(stringr,str_split)
-importFrom(stringr,str_sub)
-importFrom(stringr,str_trim)
-importFrom(tibble,as_tibble)
-importFrom(tibble,tibble)
-importFrom(tidyr,unnest)
-importFrom(tidyr,unnest_wider)
-importFrom(tidyselect,any_of)
-importFrom(tidyselect,ends_with)
 importFrom(utils,packageVersion)
-importFrom(xgboost,getinfo)

R/nflfastR-package.R

@@ -103,25 +103,40 @@
 # The following block is used by usethis to automatically manage
 # roxygen namespace tags. Modify with care!
 ## usethis namespace: start
-#' @import dplyr
 #' @import fastrmodels
-#' @importFrom cli rule
-#' @importFrom curl curl_fetch_memory
+#' @importFrom nflreadr load_pbp load_player_stats load_schedules load_rosters nflverse_sitrep
+#' @importFrom magrittr %>%
 #' @importFrom data.table setDT %between% %chin%
-#' @importFrom furrr future_map_chr future_map_dfr future_map
-#' @importFrom future plan
-#' @importFrom glue glue glue_sql
-#' @importFrom janitor clean_names
-#' @importFrom lifecycle deprecated
-#' @importFrom mgcv predict.bam
-#' @importFrom progressr progressor
-#' @importFrom rlang .data inform dots_list := .env
+#' @importFrom rlang .data inform dots_list := .env %||%
 #' @importFrom stats predict na.omit
-#' @importFrom stringr str_sub str_replace_all str_length str_extract str_detect str_trim str_remove_all str_split str_extract_all
-#' @importFrom tibble as_tibble tibble
-#' @importFrom tidyr unnest_wider unnest
-#' @importFrom tidyselect any_of ends_with
 #' @importFrom utils packageVersion
-#' @importFrom xgboost getinfo
 ## usethis namespace: end
 NULL
+
+#' @export
+nflreadr::load_pbp
+#' @export
+nflreadr::load_player_stats
+#' @export
+nflreadr::load_schedules
+#' @export
+nflreadr::load_rosters
+#' @export
+nflreadr::nflverse_sitrep
+
+## NO LONGER IMPORTED, WILL USE EXPLICIT NAMESPACING INSTEAD
+# @import dplyr
+# @importFrom cli rule
+# @importFrom curl curl_fetch_memory
+# @importFrom furrr future_map_chr future_map_dfr future_map
+# @importFrom future plan
+# @importFrom glue glue glue_sql
+# @importFrom janitor clean_names
+# @importFrom lifecycle deprecated
+# @importFrom mgcv predict.bam
+# @importFrom progressr progressor
+# @importFrom stringr str_sub str_replace_all str_length str_extract str_detect str_trim str_remove_all str_split str_extract_all
+# @importFrom tibble as_tibble tibble
+# @importFrom tidyr unnest_wider unnest
+# @importFrom tidyselect any_of ends_with
+# @importFrom xgboost getinfo

R/report.R

@@ -1,32 +0,0 @@
-#' Get a Situation Report on System, nflverse Package Versions and Dependencies
-#'
-#' @description This function gives a quick overview of the versions of R and
-#'   the operating system as well as the versions of nflverse packages, options,
-#'   and their dependencies. It's primarily designed to help you get a quick
-#'   idea of what's going on when you're helping someone else debug a problem.
-#' @details See [`nflreadr::nflverse_sitrep`] for details.
-#' @inheritDotParams nflreadr::nflverse_sitrep
-#' @inherit nflreadr::nflverse_sitrep
-#' @examples
-#' \donttest{
-#' \dontshow{
-#' # set CRAN mirror to avoid failing checks in weird scenarios
-#' old_ops <- options(repos = c("CRAN" = "https://cran.rstudio.com/"))
-#' }
-#'
-#' report(recursive = FALSE)
-#' nflverse_sitrep(pkg = "nflreadr", recursive = TRUE)
-#'
-#' \dontshow{
-#' # restore old options
-#' options(old_ops)
-#' }
-#' }
-#' @export
-report <- function(...) nflreadr::nflverse_sitrep(...)
-
-#' @export
-#' @name nflverse_sitrep
-#' @rdname report
-#' @importFrom nflreadr nflverse_sitrep
-NULL

R/top-level_scraper.R

@@ -466,7 +466,6 @@ fast_scraper <- function(game_ids,
 
 #' Load Team Rosters for Multiple Seasons
 #' @details See [`nflreadr::load_rosters`] for details.
-#' @inheritDotParams nflreadr::load_rosters
 #' @inherit nflreadr::load_rosters
 #' @seealso For information on parallel processing and progress updates please
 #' see [nflfastR].
@@ -484,7 +483,6 @@ fast_scraper_roster <- function(...) nflreadr::load_rosters(...)
 
 #' Load NFL Season Schedules
 #' @details See [`nflreadr::load_schedules`] for details.
-#' @inheritDotParams nflreadr::load_schedules
 #' @inherit nflreadr::load_schedules
 #' @seealso For information on parallel processing and progress updates please
 #' see [nflfastR].