TFS代码评论 - 显示更新的文件以响应评论
Joh*_*ohn 123 c# tfs visual-studio-2012 tfs2012 tfs-code-review
我们开始使用VS 2012和VS 2013预览中内置的代码审查功能.请求审核和添加评论看起来非常简单.如果有人添加了请求更改代码的注释,那么请求者如何进行这些更改并显示它们？
所以这个过程会像这样流动:
- 人1请求代码审查.
- 人2添加评论并选择"需要工作".
- 人1进行必要的更改.
Person 1现在如何向Person 2显示这些更改？您可以添加注释并发送它们,但文件不会更改.我假设文件来自请求原始评论时创建的变更集.第1人应该关闭此评论,并要求进行第二次审核吗？
创建第二次审核似乎不是最佳选择,因为您丢失了对话的历史记录,说明您进行更改的原因.
有很多网站展示了如何使用代码审查的基本功能,但有没有网站显示最佳做法？例如,谁应该单击文件旁边的复选框？
Cha*_*art 156
所以这个过程会像这样流动:
- 人1请求代码审查.
- 人2添加评论并选择"需要工作".
- 人1进行必要的更改.
- Person 1更新与代码审查关联的shelveset
- 人1添加评论以继续讨论
- 重复步骤2 - 5直到被接受
以下是更新与审阅相关联的搁置集所需的步骤.
- 从"代码审查"窗格中选择"查看shelveset"链接
- 从"Shelveset Details"窗格中突出显示并复制shelveset名称
- 导航到"待更改"窗格,单击"搁置"并粘贴搁置集名称
- 按下搁置集替换验证对话框上的是按钮
- 现在,审阅者可以看到更新的文件,并且审阅讨论可以继续
我已经包含了一些屏幕截图,因为我发现它有助于澄清事情.
1)从"代码审查"窗格中选择"查看shelveset"链接,如下所示:
2)从"Shelveset Details"窗格中突出显示并复制shelveset名称,如下所示:
3)导航到"Pending Changes"窗格,单击"Shelve"并粘贴shelveset名称,例如:
4)按下搁置集上的"是"按钮替换验证对话框:
- 更换搁置集不会仅触及注释文件.我们的团队成功使用此方法,我们不会删除任何评论评论. (14认同)
- 这样可行,但是如果在编辑文件时添加/删除行并替换搁置集,则该文件上的注释将指向错误的行. (9认同)
- 这个解决方案的问题在于它具有破坏性,因为所有的评论都被吹走了.我认为更好的解决方案是创建新的代码审查并将旧的代码审查设置为相关项目. (7认同)
- 太手动而不是理想的事情.TFS或Visual Studio应该有一些东西来比较连续评论之间的变化,因此评论者可以准确地看到自上次评论以来所做的更改.不存在这样的事吗？也许一个插件或什么...... (7认同)
- 我通常不愿意重复，但是@Chin的观点使这个答案完全站不住脚。是的，您可以轻松地更新您的书架，但是这样做实际上会使以前的所有评论都指向错误的文本。（因为显然在文件开头的绝对偏移量上附加了评论注释。） (2认同)
我相信正确的程序是第1人进行更改并要求进行另一次审核.当您的代码需要工作时,这意味着您将更改它,因此您需要让旧版本回顾以进行比较.如果您想查看评论,关闭后,您仍然可以在历史记录中查看旧评论.我们目前正在优化我的工作场所的代码审查流程.