Skip to content

Fix typos and Markdown formatting in the explainer docs - #321

Merged
domfarolino merged 1 commit into
webmachinelearning:mainfrom
lazizbekravshanov:fix-explainer-typos
Sep 26, 2026
Merged

domfarolino merged 1 commit into
webmachinelearning:mainfrom
lazizbekravshanov:fix-explainer-typos

Conversation

@lazizbekravshanov

Copy link
Copy Markdown
Contributor

Small editorial fixes across the README, the declarative API explainer, and the service worker explainer. No content changes.

  • README: an inline code span opens before registerTool() and never closes, so the rest of the heading renders as code.
  • Declarative explainer: a bold marker on the toolactivated and toolcanceled heading is never closed; two i.e. that should be e.g., matching the imperative example directly above them; subject and verb agreement on the TBD sentence and on submit event handlers; a stray article in "no such a tag".
  • Service worker explainer: a capital after a semicolon, a missing possessive in "the LLMs limited context window", and subject and verb agreement in "it reads/write to".

馃 Generated with Claude Code

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Copilot AI lite review requested due to automatic review settings September 26, 2026 03:57

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@w3cbot

w3cbot commented Sep 26, 2026

Copy link
Copy Markdown
Contributor

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

@domfarolino
domfarolino merged commit e7c1ef5 into webmachinelearning:main Sep 26, 2026
2 checks passed
github-actions Bot added a commit that referenced this pull request Sep 26, 2026
SHA: e7c1ef5
Reason: push, by domfarolino

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

4 participants