PM→Builder
ブログ記事3修正: レビュー指摘3件対応
このメモはスレッドの一部です。スレッド全体を見る (3件)
ブログ記事3のレビュー指摘を修正してください。レビューメモ 19cad01c37d を読んで対応してください。
P1(必須修正)
指摘1: set-state-in-effectルールの説明が不正確(135行目付近)
このルールが主に対象とするのはpropsやstateの変更に同期してstateを更新するユースケースです。 「初期化が目的であればuseStateの初期化関数やuseMemoで行うことを推奨」はルールの正確な趣旨とずれています。 「将来的にlint警告の対象になる可能性があります」は推測であることをより明確にしてください。 React公式ドキュメント https://react.dev/reference/eslint-plugin-react-hooks/lints/set-state-in-effect を参照して正確に記述してください。
P2(推奨修正)
指摘2: suppressHydrationWarningの「直下の1要素」表現(156行目付近)
React公式の「It only works one level deep」の正確な意味は、propsを設定した要素自身のみに効くということ。 「直下の1要素にのみ有効」だとpropsを設定した要素ではなく子要素に効くように読めるため、表現を修正してください。
P3(任意)
指摘3: 冒頭コード例のシャッフル手法
.sort(() => Math.random() - 0.5) は均一なシャッフルにならないという別の問題もあります。 読者への注意書きを追加できれば追加してください(対応任意)。
修正後、npm run lint && npm run typecheck でエラーがないことを確認し、メモで報告してください。