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,15 +19,19 @@ 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(data.table,"%between%")
 importFrom(data.table,"%chin%")
+importFrom(nflreadr,load_pbp)
+importFrom(nflreadr,load_player_stats)
+importFrom(nflreadr,load_rosters)
+importFrom(nflreadr,load_schedules)
 importFrom(nflreadr,nflverse_sitrep)
 importFrom(rlang,"%||%")
 importFrom(rlang,":=")

R/nflfastR-package.R

@@ -105,10 +105,22 @@
 ## usethis namespace: start
 #' @import dplyr
 #' @import fastrmodels
+#' @importFrom nflreadr load_pbp load_player_stats load_schedules load_rosters nflverse_sitrep
 #' @importFrom data.table %between% %chin%
 #' @importFrom rlang .data := .env %||%
 # We have to import something from xgboost because it is listed as dependency to
 # be able to apply models.
 #' @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

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].

pkgdown/_pkgdown.yml

@@ -37,7 +37,7 @@ authors:
   Samuel Ventura:
     href: https://twitter.com/stat_sam
   Tan Ho:
-    href: https://twitter.com/_TanHo
+    href: https://tanho.ca
   John Edwards:
     href: https://twitter.com/John_B_Edwards
 home:
@@ -128,12 +128,14 @@ reference:
     contents:
       - load_pbp
       - load_player_stats
+      - load_rosters
+      - load_schedules
   - title: Utility Functions
     contents:
       - save_raw_pbp
       - missing_raw_pbp
       - starts_with("calculate_")
-      - report
+      - nflverse_sitrep
   - title: Documentation
     contents:
       - nflfastR-package