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

一、目标和原则

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

二、流程和规则

采用Git Flow + Pull Request(PR)模式来做Code Review。

Git Flow

Java基础学习总结(158)——开发Leader如何做CodeReview_第1张图片

 

Pull Request(PR)

 

Pull Request 的说明

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

你可能感兴趣的:(Java基础学习总结,CodeReview规范,java,spring,javascript,css)