Switch from pkgdown to altdoc for documentation#503
Conversation
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
… quarto package reference Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
… navbar structure, extract _quarto.yml to vignettes/_metadata.yml Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Codecov Report✅ All modified and coverable lines are covered by tests. |
| pull_request: | ||
| types: | ||
| - opened | ||
| - synchronize | ||
| - reopened | ||
| - ready_for_review | ||
| - synchronize | ||
| - closed | ||
| paths: | ||
| - 'man/**' | ||
| - 'altdoc/**' | ||
| - 'vignettes/**' | ||
| - '.github/workflows/docs.yaml' | ||
| - 'README.md' | ||
| - 'index.qmd' | ||
| push: | ||
| tags: | ||
| - 'v[0-9]+\.[0-9]+\.[0-9]+' # build on version tags like v1.2.3 | ||
| branches: | ||
| - main | ||
| paths: | ||
| - "inst/templates/**" | ||
| - "pkgdown/**" | ||
| - "_pkgdown.*" | ||
| - DESCRIPTION | ||
| - "**.md" | ||
| - "**.Rmd" | ||
| - "man/**" | ||
| - "LICENSE.*" | ||
| - NAMESPACE | ||
| - ".github/workflows/docs.yaml" | ||
| release: | ||
| types: [published] | ||
| workflow_dispatch: | ||
| inputs: | ||
| tag: | ||
| description: Tag to deploy | ||
| required: false | ||
| default: '' | ||
|
|
||
| name: docs |
There was a problem hiding this comment.
reorder these elements to match the main branch version order as closely as possible, so the diff is easier to read
There was a problem hiding this comment.
Done in acd0e64. Reordered to match main branch: push (with branches, tags, paths), release, pull_request (with types, branches, paths), workflow_dispatch.
| pull_request: | ||
| types: | ||
| - opened | ||
| - synchronize | ||
| - reopened | ||
| - ready_for_review | ||
| - synchronize | ||
| - closed | ||
| paths: | ||
| - 'man/**' | ||
| - 'altdoc/**' | ||
| - 'vignettes/**' | ||
| - '.github/workflows/docs.yaml' | ||
| - 'README.md' | ||
| - 'index.qmd' | ||
| push: | ||
| tags: | ||
| - 'v[0-9]+\.[0-9]+\.[0-9]+' # build on version tags like v1.2.3 | ||
| branches: | ||
| - main | ||
| paths: | ||
| - "inst/templates/**" | ||
| - "pkgdown/**" | ||
| - "_pkgdown.*" | ||
| - DESCRIPTION | ||
| - "**.md" | ||
| - "**.Rmd" | ||
| - "man/**" | ||
| - "LICENSE.*" | ||
| - NAMESPACE | ||
| - ".github/workflows/docs.yaml" | ||
| release: | ||
| types: [published] | ||
| workflow_dispatch: | ||
| inputs: | ||
| tag: | ||
| description: Tag to deploy | ||
| required: false | ||
| default: '' | ||
|
|
||
| name: docs |
There was a problem hiding this comment.
Done in acd0e64. Removed duplicate name: docs on line 37.
|
|
||
| concurrency: | ||
| group: docs-${{ github.event.pull_request.number || github.ref }} | ||
| cancel-in-progress: true |
There was a problem hiding this comment.
keep the cancel-in-progress rule for concurrency
There was a problem hiding this comment.
Done in acd0e64. Added cancel-in-progress: true to the concurrency section.
…rogress Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Description
Replaces pkgdown with altdoc following the UCD-SERG/rpt pattern. Documentation renders with Quarto via
altdoc::render_docs().Changes:
altdoc,bslibto Suggests; updatedConfig/Needs/websitewithd-morrison/altdoc@recursive-qmd-searchremotequarto_website.yml: Quarto config using altdoc template variables ($ALTDOC_PACKAGE_NAME,$ALTDOC_PACKAGE_URL_GITHUB,$ALTDOC_MAN_BLOCK) with proper sections/contents structure for navbarindex.qmd: Includes README.mdCONTRIBUTING.md: Includes .github/CONTRIBUTING.mdstyles.css: Custom sidebar spacing.gitignore: Ignores altdoc/ directoryaltdoc::render_docs()instead ofpkgdown::build_site()cancel-in-progress: trueto concurrency settingsnamekey^altdoc$, removed pkgdown-specific patterns_quarto.ymlpkgdown/directory,_quarto.ymlfrom repo rootMaintained:
/preview/pr<number>/docs/directoryConfiguration:
vignettes/references.bibNote on Multi-version Documentation:
The
insightsengineering/r-pkgdown-multiversionaction is not compatible with altdoc as it is specifically designed for pkgdown sites and expects pkgdown's output structure. Multi-version documentation support was removed. Implementing multi-version docs with altdoc would require a custom workflow solution.-.testthat). - N/A for infrastructure changeFor more information see Contributing.
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.