Skip to content

Commit 384d320

Browse files
Merge pull request #425 from lorenzwalthert/fix-cran
fix CRAN issues
2 parents b1ddf7a + a93e85a commit 384d320

File tree

5 files changed

+156
-847
lines changed

5 files changed

+156
-847
lines changed

NEWS.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ editor_options:
88

99
- `parsable-R` hook now gives more context on where and how parsing
1010
fails (#423).
11+
- fixed R CMD check issues in devel
1112

1213
# precommit v0.3.1
1314

R/roxygen2.R

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,13 @@ roxygen_assert_additional_dependencies <- function() {
105105
e
106106
}
107107
)
108-
if (inherits(out, "packageNotFoundError") || ("message" %in% names(out) && grepl("Dependency package(\\(s\\))? .* not available", out$message))) {
108+
if (
109+
inherits(out, "packageNotFoundError") ||
110+
("message" %in% names(out) && (
111+
grepl("Dependency package(\\(s\\))? .* not available", out$message) ||
112+
grepl("Failed to load", out$message)
113+
))
114+
) {
109115
# case used in package but not installed
110116
rlang::abort(paste0(
111117
"The roxygenize hook requires all dependencies of your package to be listed in ",

0 commit comments

Comments
 (0)