Install DT so macros-table.qmd renders in CI

[d-morrison]

Problem

Quarto Publish has been failing on main with:

Error in `loadNamespace()`: there is no package called 'DT'
Quitting from macros-table.qmd:166-190 [macros-table]

The macros submodule's macros-table.qmd renders an interactive table via DT::datatable() / DT::JS(), but DT was not installed in any render environment, so quarto render aborts. This is a pre-existing failure (red on every recent main run), unrelated to the recent .github/ config PRs.

Fix

Add DT to the render-dependency installs:

• publish.yml and preview.yml — any::DT in the setup-r-dependencies package list (alongside knitr/rmarkdown/tibble).
• copilot-setup-steps.yml — "DT" in the install.packages() list so the Copilot agent can render locally.

DT is a render-content dependency (it comes from the macros submodule's content, not qwt's R-package code), so it lives with the other render deps rather than in DESCRIPTION.

Test plan

• This PR's preview.yml run renders the full site (including macros-table.qmd) with DT installed — the render step should now pass.
• After merge, confirm Quarto Publish on main goes green.

🤖 Generated with Claude Code

[claude]

Claude finished @d-morrison's task in 54s —— View job

---

Code Review — PR #82: Install DT so macros-table.qmd renders in CI

• Gather context and read changed files
• Review workflow changes for correctness and consistency
• Check for other workflows that render Quarto (could also need DT)
• Post review

---

Summary: looks good — minimal, correct, and well-targeted.

.github/workflows/preview.yml and publish.yml

Both add any::DT to the setup-r-dependencies package list alongside the existing any::knitr, any::rmarkdown, any::tibble. The format is consistent and the placement is correct. No issues.

.github/workflows/copilot-setup-steps.yml

Adds "DT" to the install.packages() call. This matches the existing style for that file (which uses base-R install.packages() instead of the r-lib action). Correct.

Minor observation (pre-existing, not introduced here): copilot-setup-steps.yml installs lintr, rmarkdown, and now DT, but is missing knitr and tibble that are present in both render workflows. This is a pre-existing gap and out of scope for this fix, but worth a follow-up if the Copilot agent ever needs a full render environment.

Other workflows

lint-changed-files.yaml and lint-project.yaml install only linting packages (lintr, withr, etc.) and don't invoke quarto render, so they correctly don't need DT. No gaps elsewhere.

Overall

The PR correctly diagnoses a pre-existing render failure, fixes it in all three relevant environments (publish, preview, Copilot), and correctly keeps DT out of DESCRIPTION since it's a submodule content dependency rather than a qwt package dependency. Nothing extraneous committed.

[Copilot]

Pull request overview

This PR fixes a Quarto render failure in CI caused by the DT R package not being installed, which is required by macros-table.qmd (from the macros submodule) for DT::datatable() rendering.

Changes:

• Install DT in the Publish workflow’s R dependency setup so quarto render no longer aborts.
• Install DT in the Preview workflow’s R dependency setup to keep PR preview renders consistent with Publish.
• Install DT in the Copilot agent setup workflow so the coding agent environment can render the project successfully.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
.github/workflows/publish.yml	Adds any::DT to the R dependency install list used during publish renders.
.github/workflows/preview.yml	Adds any::DT to the R dependency install list used during PR preview renders.
.github/workflows/copilot-setup-steps.yml	Adds DT to the R packages installed in the Copilot agent environment.

[github-actions]

PR Preview Action v1.8.1
Preview removed because the pull request was closed.
2026-05-26 09:32 PDT