Skip to content

Fix typos in tests and comments#423

Closed
everettbu wants to merge 4 commits into
mainfrom
fix-typos-in-tests-and-comments
Closed

Fix typos in tests and comments#423
everettbu wants to merge 4 commits into
mainfrom
fix-typos-in-tests-and-comments

Conversation

@everettbu

@everettbu everettbu commented Jan 26, 2026

Copy link
Copy Markdown

Mirror of facebook/react#35627
Original author: o-m12a


Summary

I just fixed typos as followings.

  • succesfulsuccessful
  • becuasebecause
  • enoughtenough
  • defualtdefault

How did you test this change?

This PR only includes test case description, dummy strings for test, and comments updates, so it has no impact on runtime behavior.
Therefore, I manually reviewed changed texts to ensure correctness.

@greptile-apps

greptile-apps Bot commented Jan 26, 2026

Copy link
Copy Markdown

Greptile Overview

Greptile Summary

This PR corrects 4 spelling errors across 6 files in test descriptions, comments, and test data strings.

  • Fixed succesfulsuccessful in test description
  • Fixed becuasebecause in comments (3 occurrences)
  • Fixed enoughtenough in test string data (2 occurrences)
  • Fixed defualtdefault in comment

All changes are non-functional and have no impact on runtime behavior.

Confidence Score: 5/5

  • This PR is safe to merge with zero risk
  • All changes are simple typo corrections in comments, test descriptions, and test data strings with no impact on runtime code execution
  • No files require special attention

Important Files Changed

Filename Overview
compiler/packages/babel-plugin-react-compiler/src/tests/Logger-test.ts Fixed typo in test description: succesfulsuccessful
packages/react-dom-bindings/src/server/ReactFizzConfigDOM.js Fixed typo in comment: becuasebecause
packages/react-dom/src/tests/ReactDOMFizzServer-test.js Fixed typo in test string data: enoughtenough (2 occurrences)
packages/react-dom/src/tests/ReactDOMHydrationDiff-test.js Fixed typo in comment: becuasebecause
packages/react-reconciler/src/ReactFiberCompleteWork.js Fixed typo in comment: becuasebecause
packages/react-server/src/ReactFizzServer.js Fixed typo in comment: defualtdefault

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

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

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

No files reviewed, no comments

Edit Code Review Agent Settings | Greptile

@github-actions

Copy link
Copy Markdown

This pull request has been automatically marked as stale. If this pull request is still relevant, please leave any comment (for example, "bump"), and we'll keep it open. We are sorry that we haven't been able to prioritize reviewing it yet. Your contribution is very much appreciated.

@github-actions github-actions Bot added the Resolution: Stale Automatically closed due to inactivity label Apr 26, 2026
@github-actions

github-actions Bot commented May 3, 2026

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 May 3, 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