Skip to content

polish 'Understanding WebAssembly text format'#41485

Merged
chrisdavidmills merged 3 commits intomdn:mainfrom
nichol20:nichol20-patch-2
Oct 13, 2025
Merged

polish 'Understanding WebAssembly text format'#41485
chrisdavidmills merged 3 commits intomdn:mainfrom
nichol20:nichol20-patch-2

Conversation

@nichol20
Copy link
Copy Markdown
Contributor

@nichol20 nichol20 commented Oct 12, 2025

Description

  • fix typos and minor grammatical errors
  • update the example code by correcting the size of the string in memory and adding a helpful comment (I used multi-memory.wat as a base)
  • add a helpful note for the reader who is trying to follow along

Motivation

I was reading the article and noticed a few places that could be clarified and corrected to make the article easier to follow.

Additional details

Related issues and pull requests

@nichol20 nichol20 requested a review from a team as a code owner October 12, 2025 23:19
@nichol20 nichol20 requested review from chrisdavidmills and removed request for a team October 12, 2025 23:19
@github-actions github-actions bot added Content:wasm WebAssembly docs size/s [PR only] 6-50 LoC changed labels Oct 12, 2025
nichol20 and others added 2 commits October 12, 2025 20:21
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@github-actions github-actions bot added size/m [PR only] 51-500 LoC changed and removed size/s [PR only] 6-50 LoC changed labels Oct 13, 2025
Copy link
Copy Markdown
Contributor

@chrisdavidmills chrisdavidmills left a comment

Choose a reason for hiding this comment

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

Hi @nichol20, and thank you for your contribution. Your changes look good. While I was here, I also added some minor grammatical improvements; I hope you don't mind. I basically just ran Grammarly on the article text and selected the suggestions that made sense, plus I also added in a few of my own.

@chrisdavidmills chrisdavidmills merged commit a26c067 into mdn:main Oct 13, 2025
8 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

Preview URLs

External URLs (6)

URL: /en-US/docs/WebAssembly/Guides/Understanding_the_text_format
Title: Understanding WebAssembly text format

@nichol20
Copy link
Copy Markdown
Contributor Author

Hi @nichol20, and thank you for your contribution. Your changes look good. While I was here, I also added some minor grammatical improvements; I hope you don't mind. I basically just ran Grammarly on the article text and selected the suggestions that made sense, plus I also added in a few of my own.

Thank you! I appreciate your help. Excellent work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Content:wasm WebAssembly docs size/m [PR only] 51-500 LoC changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants