TFS代码评论 – 显示更新的文件以回应评论

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

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

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

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

创build第二个审查看起来并不是最佳的，因为你失去了为什么要进行更改的谈话历史。

有很多网站展示了如何使用代码评论的基本function，但是有没有哪些网站能够展示最佳实践？ 例如，谁应该点击文件旁边的checkbox？

Answer 1

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

1. 人1请求代码审查。
2. 人2添加评论并select“需要工作”。
3. 人1进行必要的更改。
4. 人员1更新与代码审查相关的货架组
5. 人1添加评论继续讨论
6. 重复步骤2 – 5直到被接受

以下是更新与评论相关的shelveset所需的步骤。

1. 从“代码审查”窗格中select“查看搁置”链接
2. 从“Shelveset Details”窗格中突出显示并复制shelveset名称
3. 导航到“待定更改”窗格，单击“搁置”并粘贴搁置的名称
4. 按下货架上的是button更换validation对话框
5. 现在审阅者可以看到更新的文件，审查讨论可以继续

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

---

1）从“代码审查”窗格中select“查看搁置”链接，如下所示：

---

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

---

3）导航到“Pending Changes”窗格，点击“Shelve”并粘贴货架名称，例如：

---

4）按下货架上的是button更换validation对话框：

Answer 2

我相信人员1正确的程序是作出修改并要求再次审查。 当你的代码需要工作，这意味着你将会改变它，所以你会希望有旧的版本回顾比较。 如果您希望查看评论，您仍然可以在closures后的历史中查看旧评论。 目前，我们正在优化工作场所的代码审查stream程。

Answer 3

我知道这个问题很老，但是其他海报指出，这个问题仍然不被支持。 由乍得提出的解决scheme将有助于某些事情，但其他人会有奇怪的行为。

最近，TFS团队开始了一个解决scheme的计划阶段，以解决一个非常老的用户请求，以便更新一个代码审查 ，这个代码审查应该优雅地解决这个问题，使代码审查有重复。

Answer 4

你必须做两个不同的评论。 但是也有一种方法来保持历史与第二次审查。 所有你需要的是任务。

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

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

现在，在第二个审阅请求中，审阅者能够查找相关的任务，并且如果审阅者查看task1，他/她将看到changeset1和审阅请求及其评论。 所以你不会失去对话的历史。