index.md

代码评审

项目的代码是托管在公司内网的 Gitlab 上的，于是开始摸索着基于 GitLab 中项目的MR进行他人代码评审，目前代码评审流程机制及使用体验上有一些不完善的地方

• 平台无法设置 CI 不通过的情况下允许 MR合并
• 平台评论及反馈不友好，无法直观对比不同提交版本之间的差异及针对评论的修复情况
• 平台问题管理及MR绑定没有 Github 平台强大
• 平台发起 1 个MR，无法指定多人参与代码评审

代码评审目标和原则

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

解决代码冲突

如果代码评审中有任何冲突，开发人员和评审人员都应该首先根据代码开发者指南和代码评审者指南中其他文档的内容，尝试达成一致意见。
当很难达成一致时，开发者和评审者不应该在代码评审评论里解决冲突，而是应该召开面对面会议或者找个权威的人来协商。（如果你在评论里协商，确保在评论里记录了讨论结果，以便日后其他人翻阅。） 如果这样都解决不了问题，那解决问题的方式就应该升级了。通常的方式是拉着团队一起讨论、让团队主管来权衡、参考代码维护者的意见，或者让管理层来决定。不要因为开发者和评审者不能达成一致而把变更一直放在那里。

增加评审模版

评审模版应包含简介，主要变更，相关Issue/PRD

利用 Gitlab 做代码评审

常见的一些代码评审工具

• Phabircator（Facebook)
• Gerrit (Google)
• Gitlab / Github

提交代码评审之前

• 确保经过设计评审，架构和总体技术方案经过论证
• MR描述清楚业务功能，附上原型，主要变更，问题链接
• 充分自测或核心模块经过单元测试
• 代码经过 Gitlab CI 的静态检查
• 提交范围尽可能小和完整；不要一次性提交大量不相关的功能代码，把评审时间控制在20min以内
• 按照 Git Commit 提交规范进行提交
• 不阻塞他人的工作，尽快响应他人的代码评审请求
• 如果某个MR紧急，可以告知 Reviewers

一次完整的代码评审

• 提交合并请求MR
• 触发代码评审提醒和 CI pipeline 执行
• 代码评审者执行评审过程，代码评审者可以提交评论，其他人也可以参与针对问题进行持续跟进，一个MR手动设置多人参与CR
• 代码评审开发者收到评审意见，修改代码，回复评论，关闭问题
• 代码评审者确认问题是否修复，允许合并

代码评审流程

代码评审者应该关注什么

• 代码风格: 代码风格是否遵循数栈代码风格指南?
• 设计：配置、接口类的设计问题（合理性、友好性)？错误、重复的 API 调用或者封装？
• 功能性: 代码功能是否和开发者预期一致？逻辑的遗漏缺陷？
• 复杂性: 代码能不能更简单？ 其他开发者能否快速理解并在未来很容易地使用这段代码？
• 测试: 是否缺少应有的单元测试或者文档
• 命名: 开发者有没有正确地对变量、类、方法等命名？
• 文档: 开发者是否更新了相关文档？
• 注释：无用的代码或者注释

参考 代码评审者指南 获取更多信息。

挑选最适合的代码评审者

• 优先选择熟悉该业务域，技术域的开发，事先定好人选
• 指定模块最近参与修改的单个或多个同学作为 Reviewer
• 指定参与相关模块讨论和设计过的人作为 Reviewer
• 指定项目核心开发者作为 Reviewer
• 如有必要，Reviewer 可分配给多个相关人

你需要知道的MR流程

• New Merge Request

• review Merge Request

• Merge Request

开发者需遵循的工程规范

• Git Workflow：Git Workflow
• Git Commit: Git Commit
• SemVer：SemVer
• CSS 编码规范(SASS/BEM)：SASS/BEM
• BEM 使用帮助：BEM 使用帮助
• Code Style Guide(https://github.com/DTStack/Code-Style-Guide)

开发者需要知道的CI/CD流程

• Static Checking
  • ESlint
  • Type Checking
  • MDLint
  • Prettier
  • Stylelint
  • Unit Testing
• Build
  • Lock Yarn
  • Unify yarnrc
  • Migrate gitlab-runner to Jenkins

配合工具最佳

为了提升工作效率，我们可以在我们的编辑器工具中安装相关的各种插件，提升整个评审效率，由于我们大部分同学都在使用 VSCode，这里我就列举部分插件以作参考：

• ESLint 代码静态检测, 解决基本的代码风格不统一的问题，避免一些低级 bug。当然 ESLint 最好集成到开发构建环节中去
• GitLens 非常好的 git 可视化管理插件
• Gitlab MR 协助快速创建MR请求

参见

• 代码评审者指南: 代码评审者的详细指南。
• 代码开发者指南: 提交变更评审的开发者的详细指南。
• 代码评审机制保障: 代码评审机制保障