Skip to content

📖🔧:spell-check everything, in American English - #1852

Merged
openinf-commit-queue[bot] merged 1 commit into
livefrom
feat/american-english
Aug 24, 2026
Merged

📖🔧:spell-check everything, in American English#1852
openinf-commit-queue[bot] merged 1 commit into
livefrom
feat/american-english

Conversation

@DerekNonGeneric

Copy link
Copy Markdown
Member

cspell read Markdown and nothing else, so comments, templates and stylesheets
were never checked and drifted into British spelling. It now reads every file
that carries prose.

That is also the enforcement: cspell's en dictionary is the American one, so
a British variant is simply a word it does not know. No extra rule, no word
list to maintain.

What was wrong

Twenty-eight British spellings across thirteen files, in comments, in a test
name, and in two pieces of user-facing prose: /about/'s meta description and
the home page's "across the organisation".

.vscode/settings.json carried "cSpell.ignoreWords": ["behaviour"], which
existed to let one of them past. Removed with the word it protected.

Four more the dictionary accepts either way — centre, centred, coloured
and one other — do not trip cspell and were found by hand. They are the reason
this cannot be left entirely to the tool.

Scanning code found a typo in the bargain: metdata in
siteify-health-files.mts, three years of nobody reading that line.

The dictionary

Fifty-six terms added for the vocabulary a wider scan meets: Liquid keywords,
schema.org attributes, rel values, shell builtins, extension identifiers.

Forty-eight removed. OpenINF, SDK, macOS, PowerShell, changelog and
the rest were never needed — cspell's own dictionaries already know them, and
they were being whitelisted against nothing. 131 entries become 85, and what
is left is the words this project uses that no dictionary carries.

Shape

cspell moves out of verify-md.mts into verify-spelling.mts, so Markdown is
not read twice and the eighteenth verifier is about one thing. The exclusions
live in cspell.json as well as the task, so the editor extension and the
build agree on what is not ours to spell: vendored licenses, the lockfile, and
the health files generated from OpenINF/.github.

Checks

nps test passes, all 18. Verified the gate bites: a .scss comment reading
colour is now a build failure, where before it was unread. Repo swept
afterwards for the whole class — no British spelling remains outside the
verbatim license texts.

Note

Some British forms are invisible to any dictionary. spelt for spelled is
in the repository and is a real word either way; analysis is correct
American and looks British to a naive search. A checker gets most of this and
not all of it.

The spell checker read Markdown and nothing else, so comments and
templates were unread and drifted into British spelling. It now reads
every file that carries prose, and cspell's `en` dictionary is the
American one, which is what holds the project to one spelling: a
British variant is a word it does not know.

Twenty-eight of them are corrected, and the editor setting that let
`behaviour` through is gone. Four the dictionary accepts either way,
`centre` among them, were found by hand.

Scanning code found a typo in the bargain: `metdata`.

Fifty-six terms are added for vocabulary the wider scan meets, and
forty-eight removed that cspell already knew and never needed. What is
left is the words this project uses that no dictionary carries.

Signed-off-by: Derek Lewis <DerekNonGeneric@inf.is>
Assisted-by: Claude-Code:claude-opus-5
@netlify

netlify Bot commented Aug 24, 2026

Copy link
Copy Markdown

Deploy Preview for gh-pages-openinf ready!

Name Link
🔨 Latest commit 0ab46d4
🔍 Latest deploy log https://app.netlify.com/projects/gh-pages-openinf/deploys/6a8baeb097d9b00008798a9d
😎 Deploy Preview https://deploy-preview-1852--gh-pages-openinf.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@OpenINFbot OpenINFbot added the commit-queue Land this pull request when its checks pass label Aug 24, 2026
@openinf-commit-queue
openinf-commit-queue Bot merged commit d810f66 into live Aug 24, 2026
15 checks passed
@openinf-commit-queue openinf-commit-queue Bot removed the commit-queue Land this pull request when its checks pass label Aug 24, 2026
@OpenINFbot
OpenINFbot deleted the feat/american-english branch August 24, 2026 03:08
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