Skip to content

fix(cran): route font cache to tempdir under R CMD check (0.2.4)#16

Merged
milkway merged 1 commit into
mainfrom
fix/cran-cache-home
Jun 14, 2026
Merged

fix(cran): route font cache to tempdir under R CMD check (0.2.4)#16
milkway merged 1 commit into
mainfrom
fix/cran-cache-home

Conversation

@milkway

@milkway milkway commented Jun 14, 2026

Copy link
Copy Markdown
Contributor

Contexto

Resposta automática do CRAN à submissão da 0.2.3 apontou a additional issue donttest: a checagem com rede (BDR) detectou que o pacote grava no home do usuário durante os exemplos:

~/.cache/R/cardargus/Montserrat.woff2
~/.cache/R/cardargus/Open_Sans.woff2
~/.cache/R/cardargus/Roboto.woff2

Causa: install_fonts() (exemplo \donttest) e svg_to_png()/save_svg() (exemplos normais) baixam fontes do Google para o cache persistente (tools::R_user_dir("cardargus","cache")). Isso viola a política do CRAN ("não escrever no home filespace durante checks").

Correção

font_cache_dir() agora detecta R CMD check via a env var _R_CHECK_PACKAGE_NAME_ (novo helper interno running_under_check()) e usa um diretório temporário da sessão. Resultado:

  • R CMD check / exemplos / testes / vignettes → grava só em tempdir() → nada no home → CRAN limpo
  • Uso interativo / normal → cache persistente, comportamento inalterado

Verificação

devtools::check(args = c("--as-cran","--run-donttest"))
#> 0 errors | 0 warnings | 0 notes

Confirmado que ~/.cache/R/cardargus permanece inalterado após o check com --run-donttest. Teste unitário em tests/testthat/test-fonts.R valida o roteamento (78 testes no total).

Bump 0.2.3 → 0.2.4.

🤖 Generated with Claude Code

Resolve a additional issue 'donttest' do CRAN: exemplos (install_fonts,
svg_to_png, save_svg) baixavam fontes para ~/.cache/R/cardargus, violando
a política de não escrever no home durante checks.

font_cache_dir() agora detecta R CMD check via _R_CHECK_PACKAGE_NAME_
(novo helper running_under_check()) e usa um diretório temporário da sessão,
deixando o uso interativo/normal inalterado (cache persistente segue padrão).

Inclui testes (tests/testthat/test-fonts.R). Bump 0.2.3 -> 0.2.4.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@milkway milkway merged commit 4008827 into main Jun 14, 2026
1 check passed
@milkway milkway deleted the fix/cran-cache-home branch June 14, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant