Java基础学习总结（158）——开发Leader如何做CodeReview

一、目标和原则

• 提高代码质量，及早发现潜在缺陷，降低修改/弥补缺陷的成本
• 促进团队内部知识共享，提高团队整体水平
• 评审过程对于评审人员来说，也是一种思路重构的过程，帮助更多的人理解系统
• 是一个传递知识的手段，可以让其它并不熟悉代码的人知道作者的意图和想法，从而可以在以后轻松维护代码
• 可以被用来确认自己的设计和实现是一个清楚和简单的
• 鼓励相互学习对方的长处和优点
• 高效迅速完成Code Review

二、流程和规则

采用Git Flow + Pull Request（PR）模式来做Code Review。

Git Flow

 

Pull Request（PR）

 

Pull Request 的说明

• 任务完成才能提交PR
• 严禁一个PR里面有多个任务，除非它们是紧密关联的
• PR提交之后只允许针对Review发现问题再次提交代码，除非有充足的理由，严禁在同一个PR中再次提交其它任务的代码
• 提交PR时候有一个描述框，内容会自动根据Commit的message合并而成。切记，如果一次提交的内容包含很多Commit，