Skip to content

chore: fix various typos across codebase#125

Closed
everettbu wants to merge 1 commit into
mainfrom
typos
Closed

chore: fix various typos across codebase#125
everettbu wants to merge 1 commit into
mainfrom
typos

Conversation

@everettbu

@everettbu everettbu commented Dec 12, 2025

Copy link
Copy Markdown

Mirror of facebook/react#34164
Original author: JamBalaya56562


Summary

Fix various typos across the codebase to improve clarity and consistency.

Chores:

  • Correct misspellings in code comments, variable/function names, and documentation.
  • Fix typos in inline error and warning messages, test descriptions, and snapshot strings.
  • Rename identifiers to correct spelling errors (e.g., scriptIntegrity, previousEventPriority).
  • Update markdown docs and test fixture filenames to reflect proper spelling.

How did you test this change?

yarn prettier && yarn test --prod

@greptile-apps

greptile-apps Bot commented Dec 12, 2025

Copy link
Copy Markdown

Greptile Overview

Greptile Summary

This PR fixes various typos across the codebase to improve clarity and consistency. The changes include:

  • Fixed spelling errors in comments, variable names, and function names (e.g., #nextIdentifer#nextIdentifier, isFuctionExpressionisFunctionExpression)
  • Corrected misspellings in inline documentation and code comments throughout compiler and reconciler packages
  • Renamed test fixture files to fix typos in filenames (e.g., asignmentassignment, acessaccess, condtionallyconditionally)
  • Updated error code mappings in codes.json to include new error messages
  • Fixed grammar issues in comments (e.g., "able hydrate" → "able to hydrate")

All changes are non-functional and only affect readability. The typo fixes in variable/field names maintain consistency between declarations and all usage sites.

Confidence Score: 5/5

  • This PR is completely safe to merge with zero risk
  • All changes are purely cosmetic typo fixes in comments, variable names, and test fixture filenames. No functional code logic has been altered. Variable/field name corrections are consistent across declarations and usages.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
compiler/packages/babel-plugin-react-compiler/src/HIR/Environment.ts 5/5 Fixed private field name typo: #nextIdentifer#nextIdentifier and updated its usages
compiler/packages/babel-plugin-react-compiler/src/Inference/InferMutationAliasingEffects.ts 5/5 Fixed field name typo: isFuctionExpressionisFunctionExpression and comment typo: "intialized" → "initialized"
packages/react-dom-bindings/src/client/ReactFiberConfigDOM.js 5/5 Fixed multiple typos in comments: "interfer" → "interfere", "interation" → "interaction", "durationToRangeMultipler" → "durationToRangeMultiplier", "animateCallbck" → "animateCallback", "detatch" → "detach", "high-entroy" → "high-entropy", "hueristic" → "heuristic", "mistmatch" → "mismatch"
packages/react-reconciler/src/ReactFiberBeginWork.js 5/5 Fixed typos in comments: "updateFuntionComponent" → "updateFunctionComponent", "descendents" → "descendants", "passsed" → "passed", grammar fix "able hydrate" → "able to hydrate"
packages/react-dom-bindings/src/server/ReactFizzConfigDOM.js 5/5 Fixed multiple comment typos including "scriptIntegirty" → "scriptIntegrity", "seperating" → "separating", "recieve" → "receive"
compiler/packages/babel-plugin-react-compiler/src/ReactiveScopes/MergeReactiveScopesThatInvalidateTogether.ts 5/5 Fixed typos: "outupts" → "outputs", "dependncies" → "dependencies", "doesnt" → "doesn't" (with proper escaping)
scripts/error-codes/codes.json 5/5 Added new error codes 567-572 and updated error 566 to use experimental_scrollIntoView() naming

@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.

159 files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@everettbu everettbu added the Resolution: Stale Automatically closed due to inactivity label Feb 20, 2026
@github-actions

Copy link
Copy Markdown

Closing this pull request after a prolonged period of inactivity. If this issue is still present in the latest release, please ask for this pull request to be reopened. Thank you!

@github-actions github-actions Bot closed this Feb 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Resolution: Stale Automatically closed due to inactivity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants