当功能分支需要与我们的开发分支合并时,我们使用Stash并创建pull请求。我们有一个标准,每个实现的功能在开发分支中进行一次提交。我们实现这一目标的方法是在功能分支中处理功能,然后将所有提交压缩为一个。此时,我们将功能分支推送到Stash并创建一个拉取请求。
当拉取请求被拒绝,代码需要改变时,就会出现挑战。当发生这种情况时,我们在功能分支中进行所有代码更改,修改提交,强制将分支推送到Stash,并重新打开pull请求(我们不创建新的pull请求)。 Stash显示删除了1个提交,并添加了1个提交。此添加的提交包含更新的代码。但是,我们在已删除提交的代码审查中编写的所有注释都没有显示在添加的提交中。基本上,当我们查看添加的提交时,无法看到代码的哪些部分是从上一次代码审查中更新的。
我的问题是,如果有人知道在Stash中进行代码审查的更好方法并在Git中维护一个干净的提交历史记录吗?
答案 0 :(得分:1)
我们在团队中使用相同的协议。唯一的区别是我们在推送提交时不会压缩我们的提交。但是,挤压/不挤压不会解决您的问题。
每当我们的代码在创建拉取请求后需要更改时,我们都不会拒绝拉取请求,我们只是在评论中提及细节。因此,一旦开发人员进行必要的更改并将更改推送到其功能分支,拉取请求将自动更新,并且历史记录始终保留在Git中以及拉取请求的历史记录中。