Skip to content

docs: fix typo in content lifecycle-of-reactive-effects - #8566

Open
gilangabdian wants to merge 1 commit into
reactjs:mainfrom
gilangabdian:fix/typo-in-documentation
Open

docs: fix typo in content lifecycle-of-reactive-effects#8566
gilangabdian wants to merge 1 commit into
reactjs:mainfrom
gilangabdian:fix/typo-in-documentation

Conversation

@gilangabdian

@gilangabdian gilangabdian commented Jul 31, 2026

Copy link
Copy Markdown

When I was reading on the react.dev site and happened to be on the topic "Escape Hatches" in the Lifecycle of Reactive Effects content section, I accidentally found a typo in the Pitfall section. Previously it was written like this:

The linter is your friend, but its powers are limited. The linter only knows when the dependencies are wrong. It doesn't know the best way to solve each case. If the linter suggests a dependency, but adding it causes a loop, it doesn't mean the linter should be ignored. You need to change the code inside (or outside) the Effect so that that value isn't reactive and doesn't need to be a dependency.

If you have an existing codebase, you might have some Effects that suppress the linter like this:

useEffect(() => { 
  // ... 
  // 🔴 Avoid suppressing the linter like this: 
  // eslint-ignore-next-line react-hooks/exhaustive-deps
}, []);

On the next page, you'll learn how to fix this code without breaking the rules. It's always worth fixing!

typo

As you can see, the word "that" is spelled twice (that that). So, even though this seems trivial, I think typos like this can make readers feel uncomfortable.

Therefore, I'm trying to contribute to react.dev through this Pull Request. This is also my first contribution to a large open source project like React. I'm very happy 😍

@gilangabdian gilangabdian changed the title docs: fix typo in content lifecycle-of-reactive-effect docs: fix typo in content lifecycle-of-reactive-effects Jul 31, 2026
@github-actions

Copy link
Copy Markdown

Size changes

Details

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

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.

1 participant