137

我们开始使用 VS 2012 和 VS 2013 预览版内置的代码审查功能。请求审查和添加评论似乎很简单。如果有人添加评论要求更改代码,那么请求者如何进行这些更改并显示它们?

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

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

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

创建第二次审查似乎不是最佳选择,因为您丢失了有关您为什么进行更改的对话历史记录。

有很多网站展示了如何使用代码审查的基本功能,但是有没有展示最佳实践的网站?例如,谁应该点击文件旁边的复选框?

4

4 回答 4

179

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

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

以下是更新与评论关联的搁置集所需的步骤。

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

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


1)从“代码审查”窗格中选择“查看搁置集”链接,如下所示:

在此处输入图像描述


2)从“搁置集详细信息”窗格中突出显示并复制搁置集名称,如下所示:

在此处输入图像描述


3) 导航到“Pending Changes”窗格,单击“Shelve”并粘贴搁置集名称,例如:

在此处输入图像描述


4) 在搁置集替换验证对话框上按是按钮:

在此处输入图像描述

于 2014-01-14T23:17:00.370 回答
6

我相信正确的程序是由第 1 个人进行更改并要求再次审查。当您的代码需要工作时,这意味着您将对其进行更改,因此您需要回顾旧版本进行比较。如果您想查看评论,您仍然可以在历史记录中保留旧评论。我们目前正在优化我工作场所的代码审查流程。

于 2013-08-20T15:19:35.420 回答
3

我知道这个问题很老,但正如其他海报所指出的那样,它仍然不受支持。乍得提出的解决方案适用于某些事情,但对其他事情会有奇怪的行为。

最近,TFS 团队开始计划阶段,为一个非常旧的用户语音请求提供解决方案,以支持更新代码审查, 现在已迁移到 developercommunity.visualstudio,它应该通过使代码审查有迭代来优雅地解决这个问题。

于 2015-05-29T21:19:04.313 回答
0

你必须通过两个不同的评论来做到这一点。但是还有一种方法可以通过第二次审查来保留历史记录。你所需要的只是任务。

此工作流程针对基于变更集的评论进行了描述,但它也适用于基于搁置的评论。

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

现在在第二个审阅请求中,审阅者能够查找相关任务,如果审阅者查找任务 1,他/她会看到变更集 1 和审阅请求及其评论。因此,您不会丢失对话历史记录。

于 2013-10-08T07:30:45.000 回答