Add DOCX format support to pkgdown documentation #33
+123
−19
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Add docx versions to pkgdown renders
Add DOCX format support to pkgdown documentation
Jan 27, 2026
d-morrison
approved these changes
Jan 27, 2026
Codecov Report✅ All modified and coverable lines are covered by tests. |
Member
|
@copilot fix https://github.com/UCD-SERG/rpt/actions/runs/21390269803/job/61575231212?pr=33 and increment dev version; make a note in copilot-instructions to always increment dev version one past main branch |
d-morrison
requested review from
d-morrison
and removed request for
d-morrison
…n to 0.0.0.9006 Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Contributor
Author
Fixed in e8f111a. The CI failure was caused by Quarto requiring Chromium to render Mermaid diagrams when outputting to DOCX format. Added a workflow step to install Chromium before pre-rendering. Also incremented dev version to 0.0.0.9006 and added version management guidance to copilot-instructions.md. |
d-morrison
reviewed
Jan 27, 2026
Member
… copilot instructions Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Contributor
Author
Fixed in 6c19f4a. The CI failure was caused by Mermaid diagrams triggering an AssertionError when rendering to DOCX format. The Chromium process was crashing during diagram rendering. Resolved by conditionally excluding Mermaid diagrams from DOCX output using |
Copilot stopped work on behalf of
d-morrison due to an error
January 27, 2026 10:46
d-morrison
requested changes
Jan 27, 2026
| docx: | ||
| output-file: quarto_vignette.docx | ||
| pkgdown: | ||
| as_is: true |
Contributor
Author
There was a problem hiding this comment.
Yes, still necessary. The as_is: true setting tells pkgdown to include the Quarto-rendered HTML as-is without wrapping it in pkgdown's article template. This is needed because Quarto documents produce complete, self-contained HTML pages with their own styling and features. Without this setting, pkgdown would wrap the Quarto HTML in its own template, which could cause display issues with Quarto-specific elements like callouts, tabsets, and cross-references.
d-morrison
requested changes
Jan 27, 2026
Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Contributor
Author
Fixed in d341719. The CI failure was caused by intermittent errors when rendering quarto_article.qmd to DOCX format. Added retry logic to the pre-render script that attempts each format up to 2 times with a 2-second delay between retries. This should handle transient failures in the CI environment while still reporting persistent errors. 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:
|
…red files Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Contributor
Author
Fixed in d89b50e. The pre-render step succeeded, but pkgdown was trying to re-render the Quarto files during the build step, causing "Error running quarto CLI from R". Removed Quarto articles from _pkgdown.yml so pkgdown won't try to render them, and modified the workflow to copy the pre-rendered HTML files (plus RevealJS and DOCX alternate formats) to docs/articles after the pkgdown build completes. 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:
|
… during build Co-authored-by: d-morrison <2474437+d-morrison@users.noreply.github.com>
Contributor
Author
Fixed in 380a9de. The pkgdown build failed because it detected Quarto vignettes in the directory but they were missing from _pkgdown.yml (error: "2 vignettes missing from index"). Restored the Quarto articles to _pkgdown.yml and updated the workflow to temporarily move 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:
|
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Extends alternate format rendering to include DOCX (Word documents) alongside existing RevealJS slides for Quarto vignettes and articles in pkgdown documentation.
Description
When pkgdown builds the site, Quarto vignettes/articles are pre-rendered to three formats: HTML (default), RevealJS slides, and DOCX. The pre-rendered HTML files are used by pkgdown (with
.qmdsource files temporarily moved during the build to prevent re-rendering), and the HTML pages display an "Other Formats" section linking to available alternates.Changes
docxoutput format tovignettes/quarto_vignette.qmdandvignettes/articles/quarto_article.qmd.github/scripts/add-format-links.Rto detect and link both RevealJS and DOCX files (usesbi-file-wordicon for Word docs).github/scripts/pre-render-quarto.Rto render each format separately (html, revealjs, docx) instead of usingoutput_format="all"to avoid CI hanging issues. Added retry logic with up to 2 attempts per format to handle intermittent CI failures..content-visible unless-format="docx"(diagrams remain visible in HTML and RevealJS outputs).qmdfiles out of the vignettes directory during the build (preventing pkgdown from attempting to re-render them), then restore them afterward. Pkgdown uses the pre-rendered HTML files.docs/articles/after building_pkgdown.ymlfor proper indexing (pkgdown discovers all vignettes and validates they're listed in the configuration)*.docxand*-revealjs.htmltovignettes/.gitignoreDOCXanddocxtoinst/WORDLIST0.0.0.9006.github/copilot-instructions.mdTesting
devtools::document()passesdevtools::test()passes (2/2 tests)devtools::check()passes with 0 errors, 0 warnings, 0 notesspelling::spell_check_package()passesFor more information see Contributing.
Original prompt
✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.