代码评审然后呢？

Question

在我的团队中，我们有一项策略，要求至少有一个其他开发人员对每个代码进行检查。评审完成后，审阅者只需单击merge按钮，代码就会在开发中结束。最近出现了一个想法，即审查员不应该能够直接合并，而是应该将代码评审传递给开发人员的成员，并允许他们进行实际合并(2-3人)。据Scrum称，团队规模约为8人。我想得到一些反馈，这是否一个好主意，如果是的话，在什么情况下。

Answer 1

在共享分支结束之前进行所需的代码检查是一个好主意。当然，进行代码评审的人员应该是了解正在开发的系统(包括需求、体系结构和未来方向)的人，而不仅仅是任何团队成员。指派一名全新的员工进行代码评审并将其合并到一起是没有什么意义的，除非他们了解更大的背景。

不过，传递代码评审的想法对我来说没有多大意义。如果有少数更高级的开发人员，那么这些人应该是执行代码评审和合并的人，同时指导和指导较低级别的开发人员帮助他们获得更高水平的知识。随着更多的人获得经验和知识，可以赋予更多的人进行代码评审和合并软件的责任。

我不认为Scrum会给您提供任何指导，但是精益软件开发会的。语境转换和运动都是浪费，在一个精益的环境中，应该减少或消除。在当前状态下，只有一个人需要切换上下文和工作。在建议的未来状态中，有两个人需要切换上下文并在彼此之间传递消息。