search

r-lib / devtools

Tools to make an R developer's life easier
https://devtools.r-lib.org
Other
2.37k stars 755 forks source link

`check()` now checks pkgdown #2561

Open olivroy opened 4 months ago

olivroy commented 4 months ago

fix #2449

If the pkgdown config is not okay, then it errors, should it just warn?

If we decide to keep on going, we could use 

tryCatch(
  pkgdown::check_pkgdown(),
  error = function(cnd) {
    warning(conditionMessage(cnd), call. = FALSE)
  }
)

Maybe this should be pkgdown_sitrep() after pkgdown 2.1 to avoid tryCatch.

Perhaps this should happen as part of document in the build pane. If roxygenize emited some type of output about what it did r-lib/roxygen2# would help that