もうすぐ春ですね、下條です。 弊社ではプルリクエスト(以降PRと書きます)ベースのソースコードレビューのプロセスがあります。今回はソースコードレビューをする際に私が気をつけていることをまとめてみます。 レビュー方法について まずはレビュー方法についてです。 普段のレビュー 通常はGitHubなどのPRを皆が好きな時間にレビューしています。PRのコメントでレビューアにメンションを付けておくことになっているのですが、つい見逃してしまう場合があるので午後3時をレビュー時間としています。午後3時にSlackに通知が来るようになっています。 もちろん急ぎの時はお願いして急いでレビューしてもらいます。 対面 or 画面共有レビュー 対面レビューは実施コストは高いですが場合によっては有効と考えています。GitHubなどでのdiffを画面に写して一緒にレビューするというやり方が良いと思っています。私が昔勤