PRレビューは10件以上指摘したら会話しよう

はじめに

PRにコメントが大量に入っていると、GitHubは遅くなります。
稀に対象のPRページがひらけなくなります。私も、一度この状態になって困ったことがあります。

何より、コメントが大量に入っていると辛いです。

なぜ10件以上で会話が必要なのか

修正量やコメントの内容にもよりますが、成果物への否定が続くと精神的ダメージに繋がる可能性があるからです。

レビュー指摘ざっくり分類

1. ここままだと正常に動かない。致命的な問題がある。
2. ある条件において、正常に動かない。判定条件誤りなど、軽微な問題がある。
3. コーディング規約などのルールに合っていない。
4. 期待通りではあるが、今後、問題が起きる可能性がある。
5. 期待通りではあるが、可読性や保守性を考えると修正した方が良い。
6. 期待通りではあるが、「私ならこう表現するのに、勿体無い」と感じる細かい改善。

指摘数に応じて、どんな状態になるのか

😁 1〜5件

• レビュアー
  • 細かい内容でも、本人のためになるしちゃんと伝えよう。
• レビューイ
  • 細部までチェックしてくれて、ありがとう。

😃 6〜10件

• レビュアー
  • 細かい内容でも、本人のためになるしちゃんと伝えよう。
• レビューイ
  • 細部までチェックしてくれて、ありがとう。

🙂 11〜20件

• レビュアー
  • 細かい内容でも、本人のためになるしちゃんと伝えよう。
  • 類似コメントも多くなってきて、表現を省略気味で書こう。
• レビューイ
  • 細部までチェックしてくれて、ありがとう。
  • 数多いな、とりあえずやろう。
  • PRページでコメントが閉じられて、取り込み漏れが発生してくる。

😓 21〜40件

• レビュアー
  • 細かい内容でも、本人のためになるしちゃんと伝えよう。
  • 類似コメントも多くなってきて、表現を省略気味で書こう。
  • ちゃんと、確認してきたの？
• レビューイ
  • 細部までチェックしてくれて、ありがとう。
  • 数多いな、とりあえずやろう。
  • PRページでコメントが閉じられて、取り込み漏れが発生してくる。
  • ダメだった。コメント読んで対応するのに1日かかるな。

😱 41〜80件

• レビュアー
  • 細かい内容でも、本人のためになるしちゃんと伝えよう。
  • 類似コメントも多くなってきて、表現を省略気味で書こう。
  • ちゃんと、確認してきたの？
  • いい加減にして！
• レビューイ
  • あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛あ゛...........

どんな会話が必要なのか

• レビュアーから伝えること
  • どのような観点や思いで記載しているか
  • 気をつけてほしいポイント
  • 簡単に対応する方法などTipsがあれば
  • 「いつでも、悩んだら相談してね」
  • 「この対応をしてくれて、ありがとう」
• レビューイに確認すること
  • 成果物について簡単に説明をもらう
  • 作業タスクに不備、抜け漏れがなかったか

さいごに

レビュー指摘が多いと、思考停止してしまう人もいます。
心理的安全性に関する部分ですし、気持ちよく仕事していくには、思いやりを持って双方対応していくことが必要かなと考えています。