Skip to content

Fix pkgdown/bookdown CI failure by installing remotes in docs workflow - #238

Merged
iantaylor-NOAA merged 2 commits into
mainfrom
copilot/fix-pkgdown-and-bookdown-job
Aug 11, 2026
Merged

Fix pkgdown/bookdown CI failure by installing remotes in docs workflow#238
iantaylor-NOAA merged 2 commits into
mainfrom
copilot/fix-pkgdown-and-bookdown-job

Conversation

Copilot AI commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

The pkgdown-and-bookdown Actions job fails while rendering the user manual because bookdown executes devtools::install_local() from inst/bookdown/02-ex-simple.Rmd, which requires remotes at runtime. The workflow currently installs devtools and bookdown but not remotes.

  • Root cause

    • bookdown::render_book("inst/bookdown", ...) hits the Insall_Local chunk in inst/bookdown/02-ex-simple.Rmd.
    • That chunk calls devtools::install_local("../..", ...), which errors when remotes is missing.
  • Change made

    • Updated .github/workflows/update-deploy-manual-pkgdown.yaml to install remotes in the setup-r-dependencies step for pkgdown-and-bookdown.
  • Why this is minimal

    • No package code or manual content changed.
    • Single workflow dependency addition to satisfy existing render-time behavior.
- uses: r-lib/actions/setup-r-dependencies@v2
  with:
    extra-packages: any::pkgdown, local::., any::devtools, any::bookdown, any::remotes
    needs: website

Co-authored-by: iantaylor-NOAA <4992918+iantaylor-NOAA@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix the failing GitHub Actions job pkgdown-and-bookdown Fix pkgdown/bookdown CI failure by installing remotes in docs workflow Aug 11, 2026
Copilot AI requested a review from iantaylor-NOAA August 11, 2026 21:44
@iantaylor-NOAA
iantaylor-NOAA marked this pull request as ready for review August 11, 2026 21:57

@iantaylor-NOAA iantaylor-NOAA left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought that remotes was a dependency of devtools, but apparently it was moved to "suggests" 6 months ago: r-lib/devtools@713809d, making this simple change in SSMSE necessary.

@iantaylor-NOAA
iantaylor-NOAA merged commit 1066ff3 into main Aug 11, 2026
6 checks passed
@iantaylor-NOAA
iantaylor-NOAA deleted the copilot/fix-pkgdown-and-bookdown-job branch August 11, 2026 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants