Skip to content

Editorial: various typos found with cSpell - #2865

Open
nschonni wants to merge 33 commits into
w3c:mainfrom
nschonni:cspell
Open

Editorial: various typos found with cSpell#2865
nschonni wants to merge 33 commits into
w3c:mainfrom
nschonni:cspell

Conversation

@nschonni

@nschonni nschonni commented Aug 8, 2026

Copy link
Copy Markdown
Contributor

Took a pass with cSpell and cleaned up a bunch of the things that it flagged.
Could break this up a bunch so that there is a PR per spec, but I figured I'd just open the big one first and then split it if needed.

@netlify

netlify Bot commented Aug 8, 2026

Copy link
Copy Markdown

Deploy Preview for wai-aria ready!

Name Link
🔨 Latest commit 68fa9be
🔍 Latest deploy log https://app.netlify.com/projects/wai-aria/deploys/6a77610aebdc1c0008955368
😎 Deploy Preview https://deploy-preview-2865--wai-aria.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

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

@w3cbot

w3cbot commented Aug 8, 2026

Copy link
Copy Markdown

nschonni marked as non substantive for IPR from ash-nazg.

@jnurthen jnurthen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks good. Lots of these are in comments or process documents but once this is changed see no reason not to merge

Comment thread index.html
// else if (commit.message.toLowerCase().includes("typo")) return false;
// else return true;
return !/^Merge pull|^tidy|^TRAVIS|^ADMIN|^Generated by|respec|(editorial)|.md|editorial|Editiorial|edtiorial|^chore|^refactor|^tests?|^docs|w3cgruntbot|index.html|typo|^nit/i.test(
return !/^Merge pull|^tidy|^TRAVIS|^ADMIN|^Generated by|respec|(editorial)|.md|editorial|Editorial|^chore|^refactor|^tests?|^docs|w3cgruntbot|index.html|typo|^nit/i.test(

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

remove this change. This is deliberately filtering out mispellings from the chaneglog

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @nschonni

@jnurthen , since the generated changelog needs to be manually reviewed anyways I don't have a strong feeling about keeping the wrong spelling. If a commit with this creeps in I'll add it back into the publication branch.

@jnurthen
jnurthen requested review from pkra and spectranaut August 13, 2026 16:52

ghost left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks @nschonni

@daniel-montalvo daniel-montalvo changed the title chore: various typos found with cSpell Editorial: various typos found with cSpell Aug 20, 2026
@daniel-montalvo

ghost commented Aug 20, 2026

Copy link
Copy Markdown
Contributor

@nschonni Hope you don't mind me changing the PR title. It's kind of important for my own queries and I think this is more "editorial" than anything else.

@nschonni

ghost commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

@daniel-montalvo no problem. If you'd even prefer to cherry-pick and submit only certain parts, I don't mind.
If you want me to rebase and change all the typo: to Editorial: typo rename, I can do that too. I figured it's easier to review as a whole set, but would likely land as a squash-merge anyway.

@pkra
pkra removed their request for review August 22, 2026 10:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants