2017-02-03 6 views
0

QAチームの推奨事項として、プロジェクトでは1つのマスターブランチを維持しており、すべてのチームメンバーがマスターブランチに直接チェックインしています。このようにして、CIはマスター上で実行され、CIに添付されたすべてのUT、健全性ツールなどを持ち、すべてが適切に配置されているため、CIは単純です。しかし、問題はコードレビューにあります。みんながマスターに頻繁にチェックインするので、チームメンバーはコードレビューをどのように調達しますか?レビュー担当者は、特定の作者/フィーチャーコードを1つだけレビューすることは難しいです。個々の支店を持つ以外の方法はありますか?マスターブランチでのコードレビュープロセスに関する課題

+0

寄稿は、コードレビューの後に 'master' *にプッシュする必要があります。マージ後のコードレビューは意味をなさない。あなたのCIは、マージされる前にPR(例えばgithub上)をチェックするかもしれません。 – rom1v

答えて

0

各開発者は、独自のブランチを持つ必要があります。ここで、新しいコミットは、マスターにマージする前に他の開発者がコードを見直すことができます。新しいコミットはプル要求と呼ばれます。

コードレビューの古典的なツールの1つは、Gerrit:https://en.wikipedia.org/wiki/Gerrit_(software)です。 Webブラウザで新しいコミット(プルリクエスト)を視覚的に確認し、選択したブランチにマージすることができます。

よく知られているコードバージョン管理サービスを使用している場合、既にコードレビューとプルリクエスト用のツールがインストールされている可能性があります。たとえば、Bitbucketはそれを持っています。

関連する問題