Skip to content

Translate set-state-in-render lint rule - #999

Open
smikitky wants to merge 1 commit into
mainfrom
tr/eslint-set-state-in-render
Open

smikitky wants to merge 1 commit into
mainfrom
tr/eslint-set-state-in-render

Conversation

@smikitky

@smikitky smikitky commented Oct 6, 2026

Copy link
Copy Markdown
Member

set-state-in-render の翻訳です。

@smikitky
smikitky requested a review from honey32 October 6, 2026 02:57
@vercel

vercel Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Actions Updated
legacy-ja-reactjs-org Ignored Ignored Preview Oct 6, 2026 2:57am UTC

Request Review

@github-actions

github-actions Bot commented Oct 6, 2026

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! 🙌

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

サンプルが無限ループになるという技術的に誤った説明の修正が必要です。

Review effort: Balanced
Findings: 1 Low severity

Open (1)
What changed in this PR

set-state-in-render lint rule documentationを日本語化します。

Changes:

  • 見出しと解説文を翻訳
  • トラブルシューティングを翻訳
File Description
set-state-in-render.md lint ルールの日本語訳を追加

💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

```

As soon as `count` exceeds `max`, an infinite loop is triggered.
`count` が `max` を超えると、すぐに無限ループが発生します。

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

これは正しい指摘なのですが原文の問題なので原文にPRを出して修正予定です。

これで、セッタはクリックに応じてのみ実行され、React はレンダーを正常に完了し、`count` が `max` を超えることはなくなります。

In rare cases, you may need to adjust state based on information from previous renders. For those, follow [this pattern](https://react.dev/reference/react/useState#storing-information-from-previous-renders) of setting state conditionally.
まれに、以前のレンダーの情報に基づいて state を調整する必要があるかもしれません。その場合は、条件付きで state を設定する[こちらのパターン](https://react.dev/reference/react/useState#storing-information-from-previous-renders)に従ってください。

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

このドメイン付きURLは原文にPRを出して修正予定です。

This branch was successfully deployed

1 active deployment
Preview – ja-react-dev — ea8d5446 Deployed Oct 6, 2026 by vercel[bot]
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.

2 participants