Disagree respectfully in a PR review

GitHub PRTo: Tom Becker · PeerPersuadeC1
Scenario

同事 Tom 在 PR 中用 useEffect + setState 同步派生状态（由 props 计算得出的值）。你认为应该直接在渲染时计算或用 useMemo，否则会多一次渲染且容易不同步。写一条 review comment 表达不同意见，但语气要尊重、开放。

T
Tom BeckerSenior Eng · PR #482

Added a useEffect to keep `filteredRows` in sync with `rows` and `query`.

  • 1Point out the concern specifically
  • 2Explain why it matters
  • 3Suggest an alternative and stay open to discussion

Your reply