我们有一个代码审查流程,开发人员会发送一个补丁文件,其中包含对团队的更改。在审核之后,他被指示提交或进行更改并重新发送。
我们如何确保已经“承诺”的内容正是“已批准”的内容 - 即,如果他未经批准而进行后续更改并提交这些内容,我该如何检测?
我最初有'补丁文件',但是:
答案 0 :(得分:1)
最简单的方法是审阅者提交他批准的更改。我见过一些旧的时尚团队(前一段时间),只有项目经理可以提交到生产存储库。问题是,因为他必须投入太多东西,所以他没有真正检查任何东西,所以这有点毫无意义。
答案 1 :(得分:1)
最简单的解决方案是将这些补丁放到分支上,这意味着开发人员将检查分支上的代码。审阅者可以检查代码并将代码合并回特定的集成行。这样可以确保签入的代码正是已经检查过的代码。此外,这种方法具有优势,所提出的建议也将记录在版本控制中。
答案 2 :(得分:0)
审稿人也应该是一名提交人,以确保他在审查和批准时做出同样的承诺。
答案 3 :(得分:0)
我猜你可以通过在旧版本上应用补丁然后将其与最新版本的代码进行比较来做到这一点。
你可以使用svn diff。
您可能想看看这个答案:How to make svn diff produce file that patch would apply, when svn cp or svn mv was used?