Skip to content

Commit c465c6c

Browse files
CLEANUP: "Hiding" {notifier} because it is not on CRAN
1 parent 46e8534 commit c465c6c

File tree

3 files changed

+10
-10
lines changed

3 files changed

+10
-10
lines changed

DESCRIPTION

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -14,18 +14,15 @@ Suggests:
1414
tcltk,
1515
beepr,
1616
pbmcapply,
17-
plyr,
1817
progress,
19-
foreach,
2018
purrr,
21-
future (>= 1.16.0),
19+
foreach,
20+
plyr,
2221
doFuture,
22+
future (>= 1.16.0),
2323
future.apply,
2424
furrr,
25-
shiny,
26-
notifier
27-
Remotes:
28-
gaborcsardi/notifier@62d484
25+
shiny
2926
URL: https://github.com/HenrikBengtsson/progressr
3027
BugReports: https://github.com/HenrikBengtsson/progressr/issues
3128
RoxygenNote: 7.0.2

R/notifier_handler.R

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
#' Operating-System Specific Progression Feedback
22
#'
3-
#' A progression handler for [notifier::notify()].
3+
#' A progression handler for `notify()` of the \pkg{notifier} package.
44
#'
55
#' @inheritParams make_progression_handler
66
#'
@@ -18,7 +18,10 @@ notifier_handler <- function(intrusiveness = getOption("progressr.intrusiveness.
1818
## Used for package testing purposes only when we want to perform
1919
## everything except the last part where the backend is called
2020
if (!is_fake("notifier_handler")) {
21-
notifier_notify <- function(...) notifier::notify(...)
21+
if (!requireNamespace("notifier", quietly = TRUE)) {
22+
stop("Package 'notifier' is not available. See ?progressr::notifier_handler() for installation instructions")
23+
}
24+
notifier_notify <- get("notify", mode = "function", envir = getNamespace("notifier"))
2225
} else {
2326
notifier_notify <- function(...) NULL
2427
}

man/notifier_handler.Rd

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)