Skip to content

chore(r): add spelling check via usethis::use_spell_check() - #315

Merged
schloerke merged 2 commits into
mainfrom
schloerke/use-spell-check
Sep 13, 2026
Merged

chore(r): add spelling check via usethis::use_spell_check()#315
schloerke merged 2 commits into
mainfrom
schloerke/use-spell-check

Conversation

@schloerke

Copy link
Copy Markdown
Collaborator

usethis::use_spell_check() on pkg-r/:

  • spelling added to Suggests, Language: en-US in DESCRIPTION
  • tests/spelling.R — CRAN-skipped spell check test
  • inst/WORDLIST — 50 words, all jargon (Vite, jsonlite, websocket) or identifiers pulled out of code spans (ui.tsxui/tsx, pkg-jsjs, dep-discovery.Rdep, zzz.Rzzz) or names (Buie, PBC, ROR)

One real typo fixed rather than allowlisted: lowercase shiny's in default_input_handler()'s roxygen, where every other doc mention is Shiny's.

spelling::spell_check_package() reports no errors.

Adds the spelling Suggests dep, Language: en-US, tests/spelling.R, and
inst/WORDLIST (50 jargon/identifier words). Also fixes a lowercase
"shiny's" in default_input_handler()'s roxygen.
@schloerke
schloerke enabled auto-merge (squash) September 13, 2026 03:29
@schloerke
schloerke disabled auto-merge September 13, 2026 03:38
@schloerke
schloerke merged commit 54ed145 into main Sep 13, 2026
14 checks passed
@schloerke
schloerke deleted the schloerke/use-spell-check branch September 13, 2026 03:39
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