4 ポイント 投稿者 spilist2 2022-04-15 | まだコメントはありません。 | WhatsAppで共有

今回のコミット、PR、または MR で起きた「変更」について、その変更が妥当かどうかコメントを残すのは簡単ですが、変更すべきだったのにされていない部分に気づいてコメントを残すのは、コードレビューをかなり長く続けてきた今でもなお難しいと感じます。自分にとてもなじみのあるコードベースでなければ、なおさらです。

いくつか方法は思い浮かぶのですが、もっと仕組みがあるとよいと思っています。皆さんはどうしてきたのか気になります。

  • 実装するとき: あとで一緒に変更されるべき部分を、テストコードがあわせてカバーできるように実装する。
  • PR を作成するとき: PR テンプレートに、reviewee が検討できる適切な質問を追加する。(例: 今回の PR と似た目的を持つ過去の PR はありますか? その PR と比べたとき、今回の PR で何か見落としている部分はありませんか?)
    • そのコードベースで「やるべきだったのに漏れていた」ことが発生するたびに記録しておき、そのたびに PR テンプレートの該当内容を例として更新する。
  • PR をレビューするとき: レビュアーが上の質問と同じような観点で抜けている部分を探せるよう、レビューガイドに追加する。

まだコメントはありません。

まだコメントはありません。