Team Foundation Server には 3 つのレベルのユーザーがいます。
- プログラマー: コードレビュー担当者は自分のコードをレビューする必要があり、自分のコードをチェックインすることはできません
- プロのプログラマー: コードをレビューする必要はありませんが、コードのコピーを自動的にコード レビュー担当者に送信する必要があります。
- Code Reviewer : コードをレビューする必要はなく、直接チェックインできます
どうすればこれを実装できますか?
質問する
8001 次
4 に答える
5
1つのアプローチは、ブランチとアラートを使用することです。例えば...
- シニアプログラマーは、ライブコードベースにコードをチェックインできるように、「メイン」ブランチで直接作業します。
- ジュニアプログラマーは「RequiresReview」ブランチで作業し、コードレビューアはチェックインをレビューして、コードに満足した場合にのみメインにマージします。
TFSはさまざまなイベントの電子メールアラートを送信できるため、コードレビューアはチェックインイベントをサブスクライブして、コードがメインにチェックインされたとき（シニアプログラマーによって）またはレビューが必要なとき（RequiresReviewにチェックインされたとき）を知ることができます。ジュニアプログラマー）
このようにブランチを使用すると、両方向にマージする必要があります（ジュニアのチェックインをメインにマージし、シニアのチェックインをRequiresReviewにマージします）
ジュニアプログラマーのための別のアプローチは、別のブランチの代わりにShelveSetsを使用することです。このアプローチでは、ジュニアはコードをチェックインせず、コードをシェルフセットに移動してから、コードレビューアに電子メールを送信して、シェルフセットのレビューとチェックインを要求します。これにより、メインからRequiresReviewにコードをマージする必要がなくなります。ジュニアはメインブランチで「作業」しますが、チェックインはシェルフセットを介して「ゲート」されます。
はるかに統合された方法でコードレビュープロセスを管理するために購入できるサードパーティツールもいくつかあります。Web検索は、これらの種類のツールを見つけるのに役立つはずです。
編集 VisualStudio2012および2013には、コードレビューの統合サポート（基本的には上記のシェルフセットのアイデアに加えて適切なユーザーインターフェイス）が含まれているため、2010からのアップグレードは、上記の推奨される回避策よりも優れたオプションである可能性があります。
于 2012-06-03T10:12:09.460 に答える
3
これは主に TFS 2010 で構成できますが、UI から簡単に公開する方法はありません。TFS 2012 と VS2012 は、コード レビュー用のはるかに優れた UI を提供し、複雑な手順を実行できるプロセスが組み込まれています。
説明したようにソース管理のアクセス許可を構成する場合は、次のように機能します。
プログラマー
それらを TFS グループまたは AD グループに固定し、セキュリティを設定して、作業ブランチでのチェックアウトとロックのアクセス許可を持ちますが、チェックインのアクセス許可は持たないようにします。作業が完了したら、シェルフセットを作成してもらい、シェルフセット名を記載したメールをレビュアーに送信してもらいます。レビュアーは、コメント付きのセットを返すか (インラインまたは電子メールまたはワークアイテムを介して)、プログラマーに代わってコードをチェックインします。プログラマーは直接チェックインしないため、自動アラートを設定することはできません。プロセス テンプレートを変更して、タスクの状態としてレビュー準備完了を含め、タスクを移行させることができます (これは、カスタム コード レビュー ソリューションが機能する傾向がある方法です)。
コードレビュー担当者
少なくともチェックアウト、ロック、およびチェックインの権限が必要です。それらを TFS グループまたは AD グループに貼り付け、それに応じてセキュリティを設定します。シェルフセットを使用せずに直接チェックインできます。
プロのプログラマー
(この用語には注意が必要です。他のプログラマーがプロらしくないように聞こえるかもしれません。私は上級者またはベテランの方が好みです)。コード レビュー担当者と同じ権限を付与します。このグループの誰かが何かをチェックインしたときに発生する TFS アラートを構成します。コード レビュー担当者にアラートを送信します。その後、[履歴の表示] オプションを使用して変更セットを確認し、問題のプログラマーにコメントを送信できます。TFS Power Toolsを使用して、アラートの構成をより簡単に (そしてより強力に) します。
TFS プロセス テンプレートに統合された、より高度なソリューションが必要な場合は、次の 2 つのオープン ソース ソリューションのいずれかを使用できます。
または、彼らのコード (オープン ソース) を見て、このようなものを自分で構築する方法を考えてください。ただし、これは簡単なことではなく、TFS 2012 がリリースされたらおそらく役に立たなくなるでしょう。
また、コード レビューに関する TFS 2012 の新機能に関する私のプレゼンテーションもご覧ください(PPT は英語ですが、録音はオランダの会議で行われたため、オランダ語で行われました)。
于 2012-06-03T15:30:50.483 に答える
2
+1 優れたサードパーティのジェイソン 1 人はチーム レビューです
于 2012-06-03T13:08:23.813 に答える
1
Visual Studio 2012 にアップグレードできる場合は、チーム エクスプローラーの UI に優れたコード レビュー インターフェイスが組み込まれています (TFS 2012 も必要です)。見てみな：
于 2012-09-15T02:40:33.317 に答える