2017-10-27 21 views
2

shared repository modelを使用してください。要するに:github:プルリクエストなしのコードレビューですか?

  • 我々はコードを確認することができるようにしたい
  • は変わりますが、私たちは、プル要求は、私がこのようなワークフローを使用したすべての

で統合を保持する必要はありません過去にCode Collaboratorと一緒でしたが、私はそれがGit(?)とは思えませんでした。要点は、コードの変更を一度にレビューすると同時に、迅速なコード統合を可能にすることです。レビューで問題が見つかった場合、開発者は問題を修正して統合し、コードレビューを更新します。コードレビューが完了すると、「プル」(マージ)はありません。コードはすでにマージされています。レビューはただ閉じられる。

+0

実際のコードレビューが必要な場合は、ローカルリポジトリまたはリモートリポジトリのいずれかで、マージされるのを待っているrefが常に存在します。新しいコミットが証明された後でのみ、refはターゲットブランチにマージされます。コードレビューのポイントは、コードの品質を向上させることです。レビューの前にマージがあると、履歴には多くの不正なコミットが存在する可能性があります。 – ElpieKay

答えて

1

あなたのレビュープロセスにはまだconsider Gerritできます。
auto-merge during Push」もあり、レビューを実行する場所を保持しながら、すぐにマージすることができます。

これは理想的なワークフローではありませんが、重要な点はGerritを複数の方法で設定できることです。

+0

有益な答えをありがとう。私は「理想的なワークフロー」は完全に文脈上のものだと考えているだけだと言います。異なるワークフローはさまざまな状況に最適です。 –

+0

@ HawkeyeParkerあなたは正しいです、私は実際には「真の」ワークフローが1つしかないことを暗示するつもりはありませんでした。 – VonC

関連する問題