Pull Request レビューの限界｜qsona

いくつか念頭にある過去の他の方の記事や発言があるのだが、パッと見つけられないのでゼロベースで書く。 Pull Request のレビューという文化がほぼ完全に定着してからかなり経った。5年前くらいはまだ「Pull Request のレビューを開発フローに入れています」みたいなことを採用ページの文言に入れている会社が結構あったが、今時もはや当たり前になりすぎて誰も言わなくなった。 それくらい Pull Request を出してそれをチームの人がレビューするというフローは当たり前になっているが、僕はそれにずっと疑問を感じている。はっきりいってしまえば、僕は多分平均的な人の30%程度にしか Pull Request レビューに対して意義を感じていない。チームとしてコードの品質を担保したり、コードを複数人で所有するといった目的において、 Pull Request レビューが果たすことができる割合は

[p1ass]

分かる、設計レビューはPRの前に必要

[sh19910711]

2020 / "レビューの工程で「より良いコードにしていく」ことはかなり負荷が高い / むしろ、根底に関わるような重要な設計相談はそれ以前に会話なりモブプログラミングなりですべき"

[yimajo]

“ほとんど全てのコメントは単に自分の考えとして表明し、変更を要求しない。Request Changes にするのはバグがあるときだけである。”

[cuttoff19]

最小限の効用だけでもやらないより遥かにマシだと思うのと、モブプロとか普段の会話で補ってくやり方はなるほどなーというのと