Skip to content

Fix typos across documentation markdown files#496

Open
everettbu wants to merge 3 commits into
mainfrom
fix/docs-typos
Open

Fix typos across documentation markdown files#496
everettbu wants to merge 3 commits into
mainfrom
fix/docs-typos

Conversation

@everettbu

Copy link
Copy Markdown

Mirror of facebook/react#35738
Original author: him2him2


Summary

Fix 34 typos across 16 markdown documentation files, including:

  • Spelling errors (explicitlyu, errros, consdier, outweight, resister, compromised of, CreatFrom)
  • Grammar fixes (subject-verb agreement, missing articles, missing words)
  • Punctuation corrections (missing periods, capitalization, apostrophes)
  • Factual correction (OVERVIEW.md listed "5 numbers" but 7 items followed)
  • Wrong word fixes (effectaffect, it'sits, HIR formSSA form, compilercompiled)

Files changed

File Fixes
compiler/CLAUDE.md 2
compiler/README.md 1
compiler/docs/DESIGN_GOALS.md 4
compiler/docs/DEVELOPMENT_GUIDE.md 1
compiler/packages/babel-plugin-react-compiler/README.md 1
compiler/packages/babel-plugin-react-compiler/src/Inference/MUTABILITY_ALIASING_MODEL.md 7
compiler/packages/react-mcp-server/README.md 1
compiler/packages/react-mcp-server/todo.md 1
packages/dom-event-testing-library/README.md 1
packages/eslint-plugin-react-hooks/src/code-path-analysis/README.md 1
packages/react-devtools/CONTRIBUTING.md 1
packages/react-devtools/OVERVIEW.md 2
packages/react-devtools/README.md 1
packages/react-server/README.md 7
scripts/error-codes/README.md 1
scripts/release/README.md 1

Test plan

  • Changes are documentation-only (no code changes)
  • Each fix was verified against the original file content
  • No code blocks, identifiers, or technical terms were modified

Fix 34 typos across 16 markdown files including spelling errors,
grammar issues, missing words, and subject-verb agreement fixes.
@greptile-apps

greptile-apps Bot commented Feb 10, 2026

Copy link
Copy Markdown

Greptile Summary

Documentation-only PR that fixes 34 typos across 16 markdown files. Changes include spelling corrections (explicitlyu, errros, consdier, resister, CreatFrom), grammar fixes (subject-verb agreement, missing articles/words), punctuation corrections, and a factual fix in OVERVIEW.md where the text stated "5 numbers" but 7 items were listed. Notably, the DESIGN_GOALS.md fix of "HIR form" → "SSA form" corrects a meaningful technical inaccuracy in the compiler architecture documentation. All fixes are accurate and no code, identifiers, or technical terms were inadvertently modified.

Confidence Score: 5/5

  • This PR is completely safe to merge — it contains only documentation typo fixes with no code changes.
  • All 34 changes are correct typo, grammar, and factual fixes in markdown documentation files. No source code, configuration, or build files are touched. Each fix was verified against surrounding context and all corrections are accurate.
  • No files require special attention.

Important Files Changed

Filename Overview
compiler/docs/DESIGN_GOALS.md 4 fixes: "Ie" → "I.e.", "outweight" → "outweigh", missing period, and "HIR form" → "SSA form" (factual correction). All correct.
compiler/packages/babel-plugin-react-compiler/src/Inference/MUTABILITY_ALIASING_MODEL.md 7 fixes including "compromised of" → "comprised of", "errros" → "errors", "effect" → "affect", "consdier" → "consider", "it's" → "its", "CreatFrom" → "CreateFrom", and a missing period. All correct.
packages/react-devtools/OVERVIEW.md Fixes factual error ("5 numbers" → "7 numbers" to match the 7 listed items) and a missing word ("because don't" → "because we don't"). Both correct.
packages/react-server/README.md 7 fixes: "implementation" → "implementations", missing commas, "resister" → "register", "as as" → "as", missing "to", capitalization fix. All correct.

Last reviewed commit: c5ab7cd

@greptile-apps greptile-apps Bot 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.

16 files reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

@greptile-apps

greptile-apps Bot commented Feb 10, 2026

Copy link
Copy Markdown
Additional Comments (1)

compiler/README.md
Missing trailing newline

This file still appears to end without a newline (diff shows \ No newline at end of file). Many tooling/lint setups expect a trailing newline in text/Markdown files; worth adding one to avoid formatting diffs and pre-commit checks failing.

Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!

Prompt To Fix With AI
This is a comment left during a code review.
Path: compiler/README.md
Line: 1:3

Comment:
**Missing trailing newline**

This file still appears to end without a newline (diff shows `\ No newline at end of file`). Many tooling/lint setups expect a trailing newline in text/Markdown files; worth adding one to avoid formatting diffs and pre-commit checks failing.

<sub>Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!</sub>

How can I resolve this? If you propose a fix, please make it concise.

@greptile-apps greptile-apps Bot 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.

16 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants