chore: switch altdoc from recursive-qmd-search branch to main by d-morrison · Pull Request #184 · Morrison-Lab/rpt

d-morrison · 2026-08-20T02:29:13Z

The main branch of d-morrison/altdoc is 268 commits ahead of recursive-qmd-search and 0 behind. Switching to @main for both rpt and matt.contracts.

d-morrison · 2026-08-20T18:34:15Z

Working on this --- paws off until I'm done.

…rkflows Addresses three blocking issues flagged across the PR's review rounds: - docs.yaml: extra-packages block still carried unresolved `<<<<<<< Updated upstream` / `=======` / `>>>>>>> Stashed changes` markers from the altdoc branch switch, with both quarto-dev/quarto-r (correct R package) and quarto-dev/quarto (CLI repo, not installable as an R package) present. Resolved to keep d-morrison/altdoc@main and quarto-dev/quarto-r. - docs.yaml: the latest_stable_tag GITHUB_OUTPUT line was corrupted to `echo "latest_stable_mainAG" >> "$GITHUB_OUTPUT"`, breaking the stable /latest-tag/ deploy logic. Restored to `echo "latest_stable_tag=$LATEST_STABLE_TAG" >> "$GITHUB_OUTPUT"`. - copilot-setup-steps.yml: `needs: checkmain` referenced an unrecognized Config/Needs/* value (DESCRIPTION only defines Config/Needs/website). Restored to `needs: check`.

github-actions · 2026-08-20T18:38:39Z

Claude finished review — View run

All three previously-blocking issues have been fixed. Here's the review.

Code review: PR #184 (Morrison-Lab/rpt)

This is a follow-up round. The prior review (posted here, run 32343717481) verdicted Needs more work, citing three blocking issues. The PR head has since moved to a new commit (10bdb99c), and I re-fetched both affected workflow files directly at that commit to verify:

.github/workflows/docs.yaml extra-packages: block (line 143–147) — now clean: no conflict markers, single correct line d-morrison/altdoc@main, quarto-dev/quarto-r correctly retained. Fixed.
.github/workflows/docs.yaml line 85 — now reads echo "latest_stable_tag=$LATEST_STABLE_TAG" >> "$GITHUB_OUTPUT", the correct key=value form. Fixed.
.github/workflows/copilot-setup-steps.yml line 89 — now reads needs: check (unchanged from upstream). Fixed.

The diff (gh pr diff 184) is now exactly what the PR describes plus tidy supporting changes:

copilot-setup-steps.yml / docs.yaml: two one-line swaps of d-morrison/altdoc@recursive-qmd-search → d-morrison/altdoc@main, matching the PR description ("main is 268 commits ahead of recursive-qmd-search, 0 behind").
NEWS.md: a new bullet documenting the altdoc dependency switch — this satisfies the root CLAUDE.md "Add a NEWS.md bullet for any user-facing change" / news.yaml changelog-check requirement. Good addition since the prior rounds noted no entry was strictly required for CI-only changes, but including one here is harmless and arguably more correct since it documents an infra dependency change contributors may care about.
DESCRIPTION: dev version bump 1.0.2.9002 → 1.0.2.9003 (routine, matches the NEWS.md change), and Config/roxygen2/version 8.0.0 → 8.1.0 — already reviewed in the prior round (verified as a real, current CRAN roxygen2 release, and confirmed R-check-docs.yml doesn't hardcode an expected version), so re-confirmed as non-blocking.

No new issues found. No CLAUDE.md violations, no bugs, nothing scope-creeping beyond the stated intent now that the corrupted lines are gone.

Verdict

Ready for merge — all three previously-blocking issues (unresolved merge-conflict markers, corrupted GITHUB_OUTPUT line, corrupted needs: input) are fixed as of commit 10bdb99c208ed4891528d46e416a8c0346fe5815, and the diff now matches its stated scope.

Reviewed commit: 10bdb99

github-actions · 2026-08-20T18:38:40Z

💰 Cost: $0.8574 (review) — run

github-actions · 2026-08-20T18:42:07Z

PR Preview Action v1.8.1
🚀 View preview at https://Morrison-Lab.github.io/rpt/pr-preview/pr-184/
Built to branch `gh-pages` at 2026-08-20 18:41 UTC. Preview will be ready when the GitHub Pages deployment is complete.

chore: switch altdoc from recursive-qmd-search branch to main

0334fa8