waldronlab / waldronlab/bugSigSimple

Bug: pkgdown site fails to build after merging PRs into `main`

Open Beginner friendly
#50 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
R
Stars
4
Forks
18
PR merge metrics
No merged PRs in 30d

Description

Summary

After merging a pull request into main, the GitHub Actions pkgdown workflow fails and the package website is not rebuilt or deployed.

The failure occurs during the vignette rendering stage when pkgdown::build_site_github_pages() attempts to build the site.

Expected behavior

  • A PR is merged into main.
  • The pkgdown GitHub Action runs successfully.
  • The package website is rebuilt.
  • Updated documentation is deployed automatically to GitHub Pages.

Actual behavior

The pkgdown workflow fails during the Build site step, preventing the website from being generated and deployed.

Workflow error:

Error in `.f()`:
! Failed to render 'vignettes/capstoneanalysis_clare.rmd'.

Caused by error:
! Duplicate chunk label 'significant-taxa-plot'

Error log

Reading vignettes/capstoneanalysis_clare.rmd

Error in `.f()`:
! Failed to render 'vignettes/capstoneanalysis_clare.rmd'.

Caused by error:
! Duplicate chunk label 'significant-taxa-plot', which has been used for the chunk:

sig_df <- significantTaxa(
  sigs      = gut.sigs.increased,
  cn_result = cn_res,
  n         = 20,
  plot      = FALSE,
  plot_type = "lollipop",
  title     = "Taxon frequency vs. Monte Carlo threshold\n(gut, COVID-19 increased)"
)

Execution halted
Error: Process completed with exit code 1.

Additional warnings reported by pkgdown

Bootstrap 3 is deprecated; please switch to Bootstrap 5.

URLs not ok.

In DESCRIPTION, URL is missing package url
(http://waldronlab.io/bugSigSimple).

These warnings do not appear to be responsible for the workflow failure, but they may also need attention.

Reproduction

  1. Merge a pull request into main.
  2. Allow the pkgdown GitHub Action to run.
  3. Observe that the workflow fails during the vignette build stage.
  4. The package website is not rebuilt or deployed.

Possible cause

The vignette vignettes/capstoneanalysis_clare.rmd contains at least two code chunks with the same label:

{r significant-taxa-plot}

knitr requires every chunk label within an .Rmd file to be unique.

Suggested fix that i propose

  • I think we need to rename duplicate chunk labels in vignettes/capstoneanalysis_clare.rmd. OR

  • Re-run the pkgdown workflow. (But I doubt this one)

  • Optionally:

    • Migrate from Bootstrap 3 to Bootstrap 5. (Not sure about this one but we can read about it and see) OR MAY BE
    • Add the missing package URL to DESCRIPTION.

Could you be having a different approach here @lwaldron @cmirzayi

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start in vignettes/capstoneanalysis_clare.rmd and locate all chunks labeled significant-taxa-plot, then review the pkgdown GitHub Actions workflow that runs the site build. Rename duplicate labels as needed and rerun the pkgdown workflow; done means the vignette renders, the workflow succeeds, and the website is rebuilt and deployed.

Written by the indexing model from the issue text.

Assessment

Tech stack
github-actions, r
Domain
ci-cd, documentation
Issue type
Bug
Difficulty
2/5
Estimated time
1-3 hours
Activity status
Quiet
Clarity
Mostly clear
Newbie friendliness
72/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.