コードレビュー：プルリクエストフロー

Question

プルリクエストに基づいてビッグチームがコードレビュープロセスをどのように管理しますか？審査するPRの数がチーム全体のかなりの時間を要するようになると、

Answer 1

大きなチームと一緒に仕事をしている私にとってうまくいったワークフローは、チーム全体ではなくPRのために2〜3人の査読者を定義することです。

これらの人の1人は、リード開発者または設計者である必要があります。

広範なPRは、より多くのレビューアとより正式なレビュープロセスを保証する可能性があります。より小さなプッシュのためにそれを少数に保つことは、ボトルネックを少し緩めてしまうでしょう。

最も科学的な答えではありませんが、私はそれが役に立ちそうです。

Answer 2

私がお勧めすることは、いくつかのツールをプロセスに統合することです。自動コードレビューをオンラインで探すと、使用できるツールのリストが表示されます。これらはあなたのPRを自動的に分析し、あなたのためにたくさんのものを見つけます。

これは、ヒューマンコードレビューが実際に開始する前に、コードスタイル、既知のセキュリティ脆弱性、パフォーマンスや複雑さの問題などが邪魔になることを意味します。これは、時間を節約することを意味します。