コードレビュー時のコメントの意図を明確にする

コードレビューあるあるなんですけど、レビューする時に「これってなんでこうしてるんですか？」みたいなピュアなコメントを書いてしまうと、その意図がレビュイーに伝わらなくて、「詰められてる」「コードに疑念を持たれている」「修正依頼をされている」「純粋に質問されている」という解釈のブレを生んでしまって物事が円滑にいかないみたいなことは発生します。 コメントする側からするとただ単に質問しているだけなのにどうして・・・と思いがちなのですが、コードレビューに慣れていない人はこういう重要度や意図がはっきりしないコメントをたくさん書いてレビュイーを困惑させがちというのはあるんじゃないでしょうか。 重要度や意図がはっきりしないコメントを書くのが常になってしまうと、GitHubでPRを出してレビューをしてもらってマージするという開発者が何度も何度も回していくワークフローに滞りが発生してしまう、もしくはそれどころ

[emt0]

流石にみんなガラスのハート過ぎない？そんなに質問で心折れたり喧嘩するの？

[chiroruxx]

［質問］ってプレフィックスを付けても「修正しました！」みたいなレスポンスが付くからつらいんよな。

[moneycubed]

過去ひとりだけ、対面レビューで指摘したときに、言葉に詰まった上、今にも泣きそうな顔されたことある…。もうひとりのレビュワーと必死で宥めた。ので以降めちゃくちゃ気を遣っている。

[diveintounlimit]

正直ダルって思うし経験値の問題だよねとも思う。コメントはPRに向けられているのであって人に向けられているのではない。こういうののケアについてレビューする側の負荷も軽視できないと考える。

[remonoil]

なんでこうしてるんですか？って時は結局コメント文が必要になるので意図を説明するコメント文を追加してくださいって書いてる

[l__LINE__l]

テレカンした方が早い

[kuzumimizuku]

「←？」「←？？」「←わからん」「←どういう意図？」「←なんでこんな書き方してんの？」「←うるせーバカ」「←は？」「←なんだこんにゃろ」「←やるかてめえ」（最悪コードレビュー現場のレスバ例）

[wwolf]

結局フォーマルな雰囲気出そうとするから混乱するわけで、なんJのノリでやれば概ね解決しそう（小並感）

[anatoo]

書いた

[kkeisuke]

“どうやって運用していくのかについては知見を集めつつチーム内でコードレビューのやり方や規約を共有していくといいんじゃないかなと思います。”