Conversation
* Introduce new `check_urls()` function that reports on url problems with errors and call it in `check_pkgdown()` * `pkgdown_sitrep()` now calls the same functions as `check_pkgdown()` but wrapped in a helper to convert errors to messages * `check_built()` is a separate concern so it's moved to a separate file Fixes #2463. Fixes #2380.
|
@jayhesselberth do you think you might have time to take a look at this next week? |
|
with And the vignette link doesn't work, see below for the same issue. # some misconfigured yaml
url: h
articles:
- title: Customise your site
navbar: ~
contents:
- customise
- linking
-
- metadata
reference:
- title: Build
desc: Build a complete site or one of its components.
contents:
- build_site
- starts_with("build_")
- -build_site_github_pages
- init_site
-
- pkgdown_sitrep
|
|
|
|
I suspect the vignette link doesn't work because the package hasn't been installed from CRAN, and thus doesn't have a built vignette. That code comes from cli, so I think is reliable. I'll take a look at the other issue; it's not directly related to the check/sitrep unification, but I might as well fix it while I'm here. |
|
With latest changes you now see |
|
Looks good to merge. |
* Introduce new `check_urls()` function that reports on url problems with errors and call it in `check_pkgdown()` * `pkgdown_sitrep()` now calls the same functions as `check_pkgdown()` but wrapped in a helper to convert errors to messages * `check_built()` is a separate concern so it's moved to a separate file * Improve articles checking by reusing reference checks. Fixes r-lib#2463. Fixes r-lib#2380.
check_urls()function that reports on url problems with errors and call it incheck_pkgdown()pkgdown_sitrep()now calls the same functions ascheck_pkgdown()but wrapped in a helper to convert errors to messagescheck_built()is a separate concern so it's moved to a separate fileFixes #2463. Fixes #2380.