From 0dd3f15ada547f25b6122ee072116ab88b57c0bc Mon Sep 17 00:00:00 2001
From: Jared Knowles <jared@civilytics.com>
Date: Wed, 29 Apr 2026 19:25:16 -0400
Subject: [PATCH] fix(ci): use knitr::rmarkdown vignette engine + ignore
 gitea/CLAUDE
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

R CMD build (run by rcmdcheck before R CMD check) rebuilds vignettes
from source regardless of --no-vignettes. Our vignette declared
%\VignetteEngine{knitr::knitr} which requires the 'markdown' package
that isn't in CI's dependency tree. Switch to %\VignetteEngine{knitr::rmarkdown},
which uses the already-Suggests-listed 'rmarkdown' package and matches
the output: rmarkdown::html_vignette directive in the YAML header.

Also add ^\.gitea$ and ^CLAUDE\.md$ to .Rbuildignore so R CMD check
stops emitting the "hidden file" / "non-standard top-level file" notes.

Local rcmdcheck (mirroring CI's exact args) now reports
0 errors / 0 warnings / 0 notes.

The pdflatex notice in the build log is unrelated — R CMD build prints
"Not building PDF manual" and continues; with --no-manual it's silenced
entirely.
---
 .Rbuildignore                         | 2 ++
 vignettes/population-denominators.Rmd | 2 +-
 2 files changed, 3 insertions(+), 1 deletion(-)

diff --git a/.Rbuildignore b/.Rbuildignore
index 37a81c6..f06fe76 100644
--- a/.Rbuildignore
+++ b/.Rbuildignore
@@ -14,3 +14,5 @@
 ^plans$
 ^doc$
 ^Meta$
+^\.gitea$
+^CLAUDE\.md$
diff --git a/vignettes/population-denominators.Rmd b/vignettes/population-denominators.Rmd
index bb42550..fae9596 100644
--- a/vignettes/population-denominators.Rmd
+++ b/vignettes/population-denominators.Rmd
@@ -3,7 +3,7 @@ title: "Population denominators"
 output: rmarkdown::html_vignette
 vignette: >
   %\VignetteIndexEntry{Population denominators}
-  %\VignetteEngine{knitr::knitr}
+  %\VignetteEngine{knitr::rmarkdown}
   %\VignetteEncoding{UTF-8}
 ---