TFS代码评论 - 显示更新的文件以响应评论

Question

我们开始使用VS 2012和VS 2013预览中内置的代码审查功能。请求审核和添加评论看起来非常简单。如果有人添加了请求更改代码的注释，那么请求者如何进行这些更改并显示它们？

所以这个过程会像这样流动：

1. 第1个人要求进行代码审查。
2. 第2人添加评论并选择“需要工作”。
3. 第1人进行必要的更改。

Person 1现在如何向Person 2显示这些更改？您可以添加注释并发送它们，但文件不会更改。我假设文件来自请求原始评论时创建的变更集。第1人应该关闭此评论，并请求第二次审核吗？

创建第二次审核似乎不是最佳选择，因为您丢失了对话的历史记录，说明您进行更改的原因。

有大量网站展示了如何使用代码评审的基本功能，但有没有网站显示最佳做法？例如，谁应该点击文件旁边的复选框？

Answer 1

所以这个过程会像这样流动：

1. 第1人请求代码审核。
2. 人2添加评论并选择“需要工作”。
3. 人员1进行必要的更改。
4. 人1更新与代码审查相关联的shelveset
5. 第1人添加评论以继续讨论
6. 重复步骤2 - 5直到接受

以下是更新与审核相关联的搁置集所需的步骤。

1. 从“代码审核”窗格中选择“查看shelveset”链接
2. 从“Shelveset详细信息”窗格中突出显示并复制shelveset名称
3. 导航至“待更改”窗格，单击“搁置”并粘贴搁置集名称
4. 按下搁置集替换验证对话框中的是按钮
5. 现在，审阅者可以看到更新的文件，审核讨论可以继续

我已经包含了一些屏幕截图，因为我发现它有助于澄清事情。

---

1）从“Code Review”窗格中选择“view shelveset”链接，如下所示：



---

2）从“Shelveset Details”窗格中突出显示并复制shelveset名称，如下所示：



---

3）导航到“待更改”窗格，单击“搁置”并粘贴搁置集名称，例如：



---

4）按shelveset替换验证对话框上的Yes按钮：

Answer 2

我认为正确的程序是人员1进行更改并请求另一次审核。 当您的代码需要工作时，这意味着您将更改它，因此您需要让旧版本回顾以进行比较。 如果您想查看评论，关闭后，您仍然可以在历史记录中查看旧评论。 我们目前正在优化我的工作场所的代码审查流程。

Answer 3

我知道这个问题已经过时了，但其他海报所指出的仍未得到支持。 chad提出的解决方案将适用于某些事情，但对其他事情会有奇怪的行为。

最近，TFS团队开始了a very old uservoice request to enable updating of a Code Review解决方案的规划阶段，这应该通过使代码评审具有迭代来优雅地解决这个问题。

Answer 4

您必须使用两种不同的评论来完成此操作。但是，还有一种方法可以保持第二次审查的历史。你所需要的只是任务。

此工作流程适用于基于变更集的评论，但也适用于基于搁置的评论。

1. 创建任务1
2. 在检入changeset1之前，将task1添加为相关工作项
3. 检查相关的一个工作项的更改并请求审核此更改集
4. 创建任务2
5. 在检入changeset2之前，将两个任务添加为相关工作项。
6. 检查相关的两个工作项的更改并请求审核此更改集

现在，在第二次审核请求中，审核人员可以查找相关任务，如果审核人员查看了task1，他/她会看到changeset1和审核请求及其评论。所以你不会失去对话的历史。