Skip to content

perf(cran): cut check time under 10 min via skip_on_cran (v0.9.153)#109

Merged
HugoMachadoRodrigues merged 1 commit into
mainfrom
claude/cran-checktime-v09153
Jun 21, 2026
Merged

perf(cran): cut check time under 10 min via skip_on_cran (v0.9.153)#109
HugoMachadoRodrigues merged 1 commit into
mainfrom
claude/cran-checktime-v09153

Conversation

@HugoMachadoRodrigues

Copy link
Copy Markdown
Owner

The 0.9.152 incoming pre-test was OK on Windows + Debian but was held for "Overall checktime 22 min > 10 min", dominated by the test suite (~12 min on Windows R-devel).

skip_on_cran() the heavy integration / simulation / Shiny-app tests (aqp-engine fallbacks, argic designation inference, Monte-Carlo uncertainty, benchmark suite, audits, sensitivity/accuracy/edge-case/AfSP suites, and the testServer app tests). They run in full on CI (NOT_CRAN=true); the fast unit tests, the 44 canonical fixtures and the end-to-end key tests still run on CRAN.

No code or classification change. Local full R CMD check --as-cran (with vignettes + manual): testthat.R 526s → 81s; Status 1 NOTE (the local-only html-tidy artefact). Estimated Windows total ~9 min, under the 10-min limit.

🤖 Generated with Claude Code

…vy tests (v0.9.153)

The 0.9.152 incoming pre-test was OK on Windows + Debian but was held for
"Overall checktime 22 min > 10 min", dominated by the test suite (~12 min on
Windows R-devel).

skip_on_cran() the heavy integration / simulation / Shiny-app tests: the
aqp-engine fallbacks (v0986/v0989), argic designation inference (v0990), the
Monte-Carlo uncertainty suite (v0100), the benchmark suite (v09106), the
argic-films audit (v0983), the sensitivity (v0942) / accuracy (v09107) /
edge-case (v0937) / AfSP-routing (v0977) suites, and the Shiny-module testServer
tests (v09101/02/03/150, shiny-pro-servers, benchmark-levels). They run in full
on CI (NOT_CRAN=true). The fast unit tests, the 44 canonical fixtures and the
end-to-end key tests still run on CRAN.

No code or classification change. Local full R CMD check --as-cran (with
vignettes + manual): testthat.R 526s -> 81s; Status 1 NOTE (the local-only
html-tidy artefact). Estimated Windows total ~9 min, under the 10-min limit.
cran-comments updated.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings June 21, 2026 00:06

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@HugoMachadoRodrigues HugoMachadoRodrigues merged commit 8ad3cea into main Jun 21, 2026
7 checks passed
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.

2 participants