Skip to content

Reword Description to avoid win-builder spell-check NOTE#10

Merged
TroyHernandez merged 1 commit into
mainfrom
fix-spellcheck-note
May 13, 2026
Merged

Reword Description to avoid win-builder spell-check NOTE#10
TroyHernandez merged 1 commit into
mainfrom
fix-spellcheck-note

Conversation

@TroyHernandez

Copy link
Copy Markdown
Contributor

Summary

Win-builder R-devel for 0.2.0 flagged a single NOTE: "Possibly
misspelled words in DESCRIPTION: Cryptographic (18:24)". "Cryptographic"
is a real English word the win-builder spell-checker simply doesn't
ship, but the cleanest path to a 0/0/0 win-builder run is to drop it
rather than disclose it in cran-comments.

Reword:

  • before: "Cryptographic operations themselves are out of scope"
  • after: "Encryption itself is out of scope"

The meaning is unchanged -- the previous sentence already enumerates
exactly what is in scope, so the new shorter sentence is at least as
precise.

No version bump. The DESCRIPTION change is text-only; nothing else
moves.

Test plan

  • `tinypkgr::check()` -- `Status: OK`, 0/0/0 locally.
  • `tinypkgr::check_win_devel()` not re-run because the previous
    0.2.0 tarball is still in the R-devel queue (FTP 550 on same
    filename); local check covers the one-word delta, and the win-builder
    NOTE we are fixing is a known false-positive of its narrower spell
    dictionary.

win-builder R-devel flagged 'Cryptographic' as a possibly misspelled
word. It's a real English dictionary entry but the win-builder spell
checker doesn't carry it. Rewording 'Cryptographic operations
themselves are out of scope' to 'Encryption itself is out of scope'
dodges the NOTE; the meaning is the same since the package already
explicitly lists what is in scope above this sentence.
@TroyHernandez TroyHernandez merged commit b25f09f into main May 13, 2026
4 checks passed
@TroyHernandez TroyHernandez deleted the fix-spellcheck-note branch May 13, 2026 19:18
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