TFS代码评论-显示对评论的更新文件


127

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

因此,流程将如下所示:

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

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

进行第二次审核似乎不是最佳选择,因为您失去了进行更改原因的对话记录。

有大量网站显示了如何使用代码审查的基本功能,但是是否有任何网站可以显示最佳实践?例如,谁应该单击文件旁边的复选框?


+1表示“是否有任何网站可以展示最佳做法”。ALM游骑兵通常是我的深入了解的“入门者”,但是我能找到的唯一信息并不能回答您的问题。我希望看到一些使用Code Review的方法和方案。
DaveShaw

Answers:


164

因此,流程将如下所示:

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

这是更新与审阅关联的架子集所必需的步骤。

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

我提供了一些屏幕快照,因为我发现它有助于弄清楚事情。


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

在此处输入图片说明


2)在“货架详细信息”窗格中,突出显示并复制货架名称,如下所示:

在此处输入图片说明


3)导航到“待更改”窗格,单击“搁置”,然后粘贴架子集名称,例如:

在此处输入图片说明


4)按下架子上的“替换验证”对话框上的“是”按钮:

在此处输入图片说明


7
该解决方案的问题在于,它破坏性在于所有注释都被删除了。我认为更好的解决方案是创建一个新的代码审查并将旧的审查设置为相关项目。
Bardia 2014年

15
替换架子不只触摸文件注释。我们的团队成功使用了这种方法,我们不会丢失任何评论。
乍得Dienhart 2014年

10
这可行,但是如果在编辑文件并替换架子时添加/删除行,则该文件上的注释将指向错误的行。
Chin

2
我通常不愿意重复,但是@Chin的观点使这个答案完全站不住脚。是的,您可以轻松地更新您的书架,但这样做实际上会使以前的所有评论都指向错误的文本。(因为很明显,审阅注释附加在文件开头的绝对偏移量上。)
Michael Sorens,

7
太手工,不是理想的选择。TFS或Visual Studio应该可以比较连续审阅之间的更改,因此审阅者可以准确看到自上次审阅以来所做的更改。不存在这样的东西吗?也许是插件之类的东西
–Iúridos Anjos

6

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


2

我知道这个问题很旧,但正如其他张贴者所指出的那样,它仍然不受支持。乍得提出的解决方案适用于某些事物,但对于其他事物则具有奇怪的行为。

最近,TFS团队开始了一个非常老的用户语音请求解决方案的规划阶段,以启用代码审查的更新,这应该通过使代码审查具有迭代性来优雅地解决此问题。


1
大。我也会很期待。我欢迎一个真正的解决方案。我的帖子只是解决问题的一种方法。
乍得Dienhart,2015年

希望它不会看起来像天蓝色的DevOps拉取请求。他们是一场噩梦。
StingyJack

0

您必须使用两个不同的评论来执行此操作。但是,还有一种方法可以保留第二次审核的历史记录。您所需要的只是任务。

描述了此工作流用于基于变更集的审阅,但它也适用于基于搁置的审阅。

  1. 创建任务1
  2. 在检入changeet1之前,将task1添加为相关工作项
  3. 检入与一个工作项目相关的变更,并请求对此变更集进行审查
  4. 创建任务2
  5. 在检入changeet2之前,将两个任务都添加为相关工作项。
  6. 检入与两个相关工作项相关的变更,并请求对此变更集进行审查

现在,在第二个审阅请求中,审阅者能够查找相关任务,并且如果审阅者寻找任务1,他/她将看到更改集1和审阅请求及其注释。这样您就不会丢失对话历史记录。

By using our site, you acknowledge that you have read and understand our Cookie Policy and Privacy Policy.
Licensed under cc by-sa 3.0 with attribution required.