積み重ねられた PR を使用してコード レビューを再考する この投稿ではシフトレフトの原則を検討し、スタックされた PR がますます有用になることを示唆しています。 ピアコードレビュープロセスはソフトウェア開発の重要な部分です。ソフトウェアの品質を維持し、標準、プロジェクト要件、スタイル ガイドへの準拠を促進し、学習と知識の伝達を促進します。 コードレビューの有効性 十分に小さなコード変更をレビューする場合は有効性が高くなりますが、変更のサイズが大きくなると有効性は指数関数的に低下します。効果を発揮するために必要な精神的な集中力を維持するには、大規模なコード レビューを行うのは大変な作業です。 通常、レビュー期間が長くなるほど、全体的なレビューの効果は低下します。 では、なぜプル リクエスト (PR) のサイズを制限できないのでしょうか?多くの変更は小さく始めることができますが、小さな 2 行の変更が突然、レビュー担当者との何度もやり取りを含む 500 行のリファクタリングに成長する可能性があります。 一部のエンジニアリング チームは、作業を継続しながら長期間実行される機能ブランチを維持するため、レビューが困難になります。 では、どのようにして適切なバランスをとればよいのでしょうか?単純。スタックされた PR を使用します。 スタックされた PR とは何ですか? スタックされたプル リクエストは、単一のプル リクエストに大規模なモノリス変更をバンドルするのではなく、小規模な反復的な変更を加え、互いに積み重ねられます。スタック内の各 PR は 1 つの論理変更のみに焦点を当てているため、レビュー プロセスが管理しやすくなり、時間が短縮されます。 また、私たちは昨年、このヘルプが ように表現するかを説明する記事を書きました。 コードの変更をファイルや機能ごとに分類するのではなく、物語としてどの PR をスタックする理由 より効果的なコードレビューの文化を構築する以外にも、スタック型 PR にはいくつかの利点があります。 初期のコードレビューのフィードバック 大規模な機能を実装していると想像してください。機能全体を作成してからコードレビューをリクエストするのではなく、最初のフレームワークを切り出し、すぐにフィードバックに出すことを検討してください。 これにより、設計に関するフィードバックを早期に取得できるため、膨大な時間を節約できる可能性があります。 CI フィードバック サイクルの高速化 スタックされた PR は、変更が継続的に統合およびテストされるため、 実践をサポートし、問題の早期検出と修正が可能になります。 シフトレフトの 変更は、問題を早期に発見するために少しずつマージされますが、本番環境がダウンしないことを祈りながら 1 つの大きな変更をマージするのではありません。 知識共有 コードレビューは後世にとっても素晴らしいものです。コードの変更は、機能の実装の背後にある思考プロセスを物語るものであるため、変更の内訳によって、より効果的な知識の伝達が行われます。 チームメンバーが変更を理解しやすくなり、将来に向けた知識の共有が促進されます。 ブロックされない状態を維持する コードのレビューと承認を待つのは、イライラするプロセスになる可能性があります。スタックされた PR を使用すると、開発者はレビュー担当者が以前の PR を承認するのを待たずに、機能の複数の部分に取り組むことができます。 キャッチは何ですか? では、なぜもっと多くの開発者がコードレビューにスタックされた PR を使用しないのでしょうか? この積み重ねられた PR ワークフローは、コード レビューを管理しやすくし、開発者の生産性を維持するという望ましい実践の両方に対応していますが、残念ながら、Git または GitHub のどちらでもネイティブに十分にサポートされていません。 その結果、エンジニアがこのスタッキング技術を既存の Git および GitHub プラットフォームに組み込めるように、オープンソース コミュニティ全体で が開発されました。しかし、PR を積み重ねることは話の一部にすぎません。 いくつかのツール 更新中 コードレビューのフィードバックを受け取り、スタックの一部に変更を加えると、後続のすべてのブランチでリベースして競合を解決する必要があります。 例を挙げてみましょう。スキーマの変更、バックエンドの変更、フロントエンドの変更を必要とする変更に取り組んでいることを想像してください。 これにより、最初に簡単なスキーマ変更をレビュー用に送信し、それがレビューされている間にバックエンドとフロントエンドでの作業を開始できるようになりました。スタックされた PR を使用すると、これら 3 つの変更すべてを 3 つの異なるレビューでレビューできます。 この場合、次のようなスタックが作成される可能性があります。 ここで、 、 、 は、互いに積み重ねられた 3 つのブランチを表します。 demo/schema demo/backend demo/frontend ここまでは理にかなっていますが、新しいコミットの作成が必要なスキーマ変更に関するコード レビュー コメントを受け取った場合はどうなるでしょうか?突然、コミット履歴は次のようになります。 ここで、後続のすべてのブランチを手動でリベースし、すべての段階で競合を解決する必要があります。 10 個のスタックされたブランチがあり、競合を 10 回解決する必要がある場合を想像してください。 結合 しかしそれだけではありません。PR をスタックにマージするのは本当に悪夢になる可能性があります。 PR をマージするには、 、 、および の 3 つのオプションがあります。それぞれの舞台裏で何が起こっているのかを理解してみましょう。 squash merge rebase コミットの場合、Git は PR の既存のすべてのコミットから変更を取得し、それらを 1 つのコミットに書き換えます。この場合、それらの変更がどこから来たのかに関する履歴は保持されません。 squash コミットは、2 つ以上のコミットの組み合わせで表される特別なタイプの Git コミットです。つまり、 コミットと非常によく似た動作をしますが、その親に関する情報もキャプチャします。一般的なシナリオでは、マージ コミットには 2 つの親があります。ベース ブランチ (PR がマージされる) の最後のコミットと、マージされたフィーチャー ブランチの最上位のコミットです。 merge squash このアプローチではコミット履歴により多くのコンテキストが与えられますが、望ましくない が誤って作成されてしまいます。 非線形の git 履歴 最後に、 とマージの場合、Git はコミットをベース ブランチに再書き込みします。したがって、 コミット オプションと同様に、元のコミットに関連付けられた履歴はすべて失われます。 rebase squash 通常、PR をスタックするときに コミット戦略を使用している場合、作業は少し簡素化されますが、ほとんどのチームは、git 履歴をクリーンに保つためにその戦略を使用することを推奨しません。つまり、 または マージのいずれかを使用している可能性があります。 merge squash rebase これにより、後続のマージされていないスタックされたブランチすべてに対してマージ競合が発生します。 上の例では、最初のブランチの メインラインにスカッシュ マージするとします。 と の変更を含む新しいコミット が作成されます。 demo/schema A1 A2 D1 Git は どこから来たのかを知らず、 はまだ に基づいているため、 メインラインの上にリベースしようとするとマージ競合が発生します。 D1 demo/backend A2 demo/backend 同様に、 リベースした後に をリベースすると、同じ問題が発生します。したがって、10 個のスタックされたブランチがあり、そのうちの 1 つをマージした場合、これらの競合を 9 回解決する必要があります。 demo/frontend demo/backend 私たちはまだ表面をなぞったばかりです。コミットの並べ替え、ブランチの分割、折りたたみ、名前変更など、 あり、スタックされた PR を処理するときに管理するために膨大なオーバーヘッドが発生する可能性があります。 他にも多くのユースケースが そのため、Aviator の一部としてスタック型 PR 管理を構築しました。 Aviator CLI が異なる理由 Aviator は、既存のツールの上に位置する拡張レイヤーと考えてください。 Aviator は、GitHub、Slack、Chrome、および Git CLI に接続して、強化された開発者エクスペリエンスを提供します。 Aviator CLI は他のすべてのものとシームレスに動作します。 CLI は Git 上の単なるレイヤーではなく、GitHub 全体のスタックのコンテキストも理解します。例を考えてみましょう。 スタックの作成 スタックの作成は非常に簡単です。この場合を除き、 CLI を使用してブランチを作成し、スタックが確実に追跡されるようにします。たとえば、スキーマ ブランチと対応する PR を作成するには、次の手順に従います。 av av stack branch demo/schema # make schema changes git commit -a -m "[demo] schema changes" av pr create Aviator は GitHub にも接続されているため、スタックを簡単に視覚化できます。 または、ターミナルから視覚化したい場合でも、CLI コマンドを使用して実行できます。 スタックの更新 スタックの使用は簡単になります。任意のブランチに新しいコミットを追加し、スタック内のどこからでも を実行するだけですべてのブランチを同期できます。 Aviator はすべてのブランチを自動的にリベースするため、実際にマージ競合が発生した場合は、一度解決するだけで済みます。 av stack sync スタックのマージ これは、Aviator ツールが既存のツールよりも優れている点です。 Aviator では、数千もの変更の自動マージを大規模に管理するために、最も高度な MergeQueue の 1 つを構築しました。 Aviator は、CLI およびスタックされた PR とのシームレスな統合をサポートしています。したがって、PR の部分スタックまたは完全スタックをマージするには、CLI を使用するか、GitHub にコメントを投稿することで、PR を Aviator MergeQueue に割り当てることができます: 。 av pr queue /aviator stack merge Aviator は、キューに入れられたすべてのスタックの検証、更新、自動マージを順番に自動的に処理します。 PR がマージされたら、今度は を実行してすべての PR を更新し、マージされたすべての PR を消去します。 av stack sync --trunk シフトレフトが未来だ スタックされた PR は、変更をより小さな部分に分割する必要があるため、最初はより多くの作業のように思えるかもしれません。ただし、コード レビューの効率の向上、フィードバック ループの高速化、学習の機会の強化は、確実にこのオーバーヘッドを上回るでしょう。 私たちがシフトレフトの原則を受け入れ続けるにつれて、スタック型 PR はますます有用になるでしょう。 Aviator CLI は、スタックされた PR をはるかに簡単に管理するための優れた方法を提供します。 CLI は で完全に無料です。ぜひお試しいただき、 でフィードバックを共有していただければ幸いです。 オープンソース ディスカッション掲示板 Aviator では、開発者がより速く、より優れたものを構築できるように、第一原則に基づいて開発者生産性ツールを構築しています。
