PRレビューってかなり壊れた仕組み
私はもうずっとPRレビューってかなり壊れた仕組みだと思っているので、下のポストはすごく気持ちがわかる
PRコメント対応も立派なタスクでそれなりの時間を要するのだが、他のタスクと違って優先度を決めるプロセスを経ていないのが大きな問題
原則としてタスクの優先度を決定するのはPOなりPMであるべきで、PRレビューワーであってはならない。別タスクになれば、PO/PMが的確な優先度をつけられる
PRレビューワーがコメントを通して書き換えるタスクを与えると、それをPO/PMを経ずに優先度を決定していることになる
PRレビューワーには本来そんな権限はない。自分の趣味のスタイルやほぼ起こり得ないエッジケースのために、貴重なリソースを無駄にしている可能性も高い
PRレビューワーが拒否権を持つのは、セキュリティホールとクリティカルな問題を防止するためである。他のところで乱用してはならない
でも実際にはレビューワーは結構面倒なコメントを書くことが多い。そして拒否権を持っている。自分の要望を事実上最優先にしてしまう
しかもレビューワーはPMやPOと違って、遅延に直接責任を持たない。一番遅延を発生させる人物なのに責任がない状態が起こりうる
もちろんコメント対応がちょちょッとできて、すぐに対応できるなら問題ないだろう。またリアルタイムに近い感じでレビューが返せるのなら、それも負担は少ない
でもそうじゃないのなら、レビューワーは自分の権限をよく考えないといけない。また負担軽減を考えないといけない