TFS 代码审查-根据注释显示更新的文件

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

这个过程是这样的:

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

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

创建第二次审查看起来并不是最佳选择，因为你会丢失关于为什么要做出改变的谈话历史。

有大量的网站展示了如何使用代码评审的基本功能，但是有没有一些网站展示了最佳实践呢？例如，谁应该单击文件旁边的复选框？

45471 次浏览
小开

我相信正确的程序是人1做出修改并要求另一个审查。 当您的代码需要工作时，这意味着您将对其进行更改，因此您将希望回过头来查看旧版本以进行比较。 如果您希望查看评论，您仍然可以在历史记录中看到关闭后的旧评论。 我们目前正在我的工作场所优化我们的代码审查过程。

小开

你必须用两种不同的评论来做这件事。但是，还有一种方法可以保留第二次审查的历史。你需要的只是任务。

这个工作流描述了基于变更集的评审，但是它也适用于基于搁置的评审。

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

现在，在第二个评审请求中，评审员可以查找相关的任务，如果评审员查找 task1，他/她将看到 changeset1和评审请求及其评论。所以你不会失去谈话记录。

小开
最佳答案

这个过程是这样的:

  1. 个人1请求代码审查。
  2. 人2添加注释并选择“需要工作”
  3. 人1进行必要的更改。
  4. 个人1更新与代码审查相关联的搁置集
  5. 参与者1添加注释以继续讨论
  6. 重复步骤2-5，直到被接受

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

  1. 从“ CodeReview”窗格中选择“ view shelveset”链接
  2. 从“搁置集详细信息”窗格突出显示并复制搁置集名称
  3. 导航到“ Pending Changes”窗格，单击“ Shelve”并粘贴搁置集名称
  4. 按搁置集替换验证对话框上的 Yes 按钮
  5. 现在评审员可以看到更新的文件，评审讨论可以继续

我已经包括一些屏幕截图，因为我发现它有助于澄清的东西。

1)从“代码审查”面板中选择“视图搁置集”链接，如下所示:

enter image description here

2)从“搁置集详细信息”窗格高亮显示并复制搁置集名称，如下所示:

enter image description here

3)导航到“挂起的更改”面板，点击“搁置”并粘贴搁置集名称，例如:

enter image description here

4)按下搁置集替换验证对话框上的“是”按钮:

enter image description here

小开

我知道这个问题很老了，但是它仍然没有得到其他海报所指出的支持。乍得提出的解决方案对某些东西有效，但对另一些东西会有奇怪的行为。

最近，TFS 团队开始了 一个非常古老的用户语音请求来支持代码审查的更新 现在迁移到 developercommunity Visual Studio解决方案的规划阶段，这个解决方案应该通过让代码审查进行迭代来优雅地解决这个问题。


提交答案