Prep 2.0.1: NEWS, docs, CRAN checklist fixes - #297
Conversation
NEWS.md's "APCalign 2.0.0" section had kept accumulating bullets after the CRAN 2.0.0 submission instead of starting a new heading. Split it: the six post-submission bullets move to a new "APCalign 2.0.1" section, joined by two that were missing (#295 fuzzy_match qualifier fix, #293 internal gsub-helper dedupe). The original six 2.0.0 bullets are unchanged. Also fixes a devtools::document() warning: match_taxa.R's @details linked to [apply_match()], an internal @nord helper with no doc topic to resolve to. Ref #296 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- .Rbuildignore: exclude AGENTS.md and CITATION.cff from the built package. Both are repo-root files R CMD check doesn't expect (CITATION.cff is GitHub's citation format, unrelated to R's own inst/CITATION mechanism) — was producing 2 NOTEs. - DESCRIPTION: Language en -> en-GB. The package consistently uses British/Australian spelling (standardise, organisations, labelling, favour, behaviour, ...); the ambiguous "en" was defaulting to en-US and flagging all of them as misspellings. - inst/WORDLIST: whitelist the remaining devtools::spell_check() hits — proper nouns (AusTraits, APNI, ORCID, GBIF, ...), state/territory abbreviations (NSW, Qld, ChI, ...), taxonomic jargon (infraspecific, subsp, aff, ...), and species epithets used in examples/tests. None were real typos. devtools::check() now 0 errors, 0 warnings, 0 notes locally, and devtools::spell_check() passes clean. Ref #296 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Code quality / CRAN policy pass on the 2.0.1 checklist (items that don't require live API access to verify): - default_version() and get_versions() call the GitHub releases API directly in their @examples, unwrapped -- every R CMD check with --run-donttest (devtools::check()'s default) was hitting that API live. Wrapped both in \donttest{}, matching every other network-dependent example in the package. - Replaced remaining T/F shorthand with TRUE/FALSE in user-facing example code (README.Rmd, vignettes/APCalign.Rmd) and tests (test-state_diversity.R). R/ itself was already clean. - Added skip_if_offline() to test-connection.R's "functions return visibly when online" test, alongside its existing skip_on_ci()/ skip_on_cran(), so it degrades gracefully under a genuine local outage too, not just in CI/CRAN. - Confirmed clean: devtools::document() (NAMESPACE/man up to date), no print()/cat() outside message(), no <<-, no unrestored options()/par()/setwd(), no tests writing outside tempfile()/ tools::R_user_dir(). - DESCRIPTION version bumped 2.0.0 -> 2.0.1. - cran-comments.md rewritten for the 2.0.1 patch release. Ref #296 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
Working detail, commit by commit: 0be5483 — NEWS.md / roxygen fix 051f530 — check NOTEs / spell_check b4ca39b — donttest / T-F / version bump Known open item: |
1 participant
Works through the Documentation, Code quality, and most of Package
checks / CRAN policy / Version & metadata in the 2.0.1 pre-submission
checklist.
Ref #296
🤖 Generated with Claude Code