コードレビューをどうすれば早くできるかの悩み

necocoa
·

Four keys の指標にもある 変更のリードタイム、これをいかに早くできるか、に悩んでいる。

直近で課題に感じているのはプルリクエストを出してからレビューに入るまでの時間が長い。いくつか要因はあるものの、簡易でわかりやすいプルリクエストでもレビューされずに溜まっていることがある。これはレビューの優先順位が低い(見てない)状態になっていそう。

レビューがされないからいつまでも脳内メモリを圧迫されたり、1日後のレビューで修正があって、後続の対応に影響がある、など生産性を明らかに下げている原因になっていそう。

変更の内容がわからず、プルリクエストの内容が正しいのかわからない → 他の人に任せる(スルー)→ プルリクエストをそもそも見なくなる。みたいな循環になっているパターンが見受けられ、それをどうすると解消できるか、を考えていた。

施策の1つとして、同期的に書いた人含め、ペア/モブ・レビューをするのがレビューのハードルを下げて、進めていくコツを掴むものなのかもと思って導入してみる。

雑に話してみると物事は進むのだ。

@necocoa
エンジニアリングマネージャーのひとりごと