😵‍💫

PRレビューと自分の作業、どっちを優先する?

に公開

はじめに

PRレビュー後回しになりがち問題ありますよね?(無い?)
最近では、AIがコードを書くことによって、より多くのコードが書かれて、よりコードレビューの機会が増えているチームもあるかもしれません。

大切なこと

リリースして初めて成果になる。これはとても大切な考え方だな〜と思ってます。
コードを書いてPRを出すだけでは何の成果にもなりません。これはPRを作っただけ。
マージされて、リリースすることで、初めて価値を生みます。

でも、やることたくさんある

とは言っても、みんな日々たくさんのタスクを抱えていて、忙しいはず。
限られた時間の中で、自分の作業を早く進めたい気持ちもめちゃくちゃよくわかります。

ただ、チー厶の成果を考えたとき、レビューを後回しにするのは、価値を生むのを先延ばししていてめちゃくちゃ勿体無いです。

価値を生まない状況

AさんがPR1作成 → レビュー待ち
BさんがPR2作成 → レビュー待ち
CさんがPR3作成 → レビュー待ち
..

みんな「いつレビューされるのかなぁ」とモヤモヤしそうです。
レビュアーの状況を伝えると、多少マシかもしれません。

さっさとレビューしてしまう方が、あなたの作業は多少遅れますが、より早くリリースできて、より早くチームの成果になります。

AさんがPR1作成 → すぐレビューされてリリース
BさんがPR2作成 → すぐレビューされてリリース
CさんがPR3作成 → すぐレビューされてリリース

まずは「ざっと見る」もアリでは?

PRが大きくて(or 変更内容が複雑で)、レビューに時間が掛かりそうなとき、レビューしようと思っても、腰が重かったり、全部ちゃんと見る時間無いよー😭 ということもあると思います。

そんなときは、いったん気になった点だけでも返しておくと良いと思います。
レビュイーがそれに対して修正したり、検討したり、何かしらアクションすることができるので。

最後に

いつもできているわけではないですが、レビューの優先度は意識的に上げるようにしています。

Discussion