基本原则
-
尊重PR提交者
-
代码首先是给人看的 ，然后才是让计算机运行的
-
IDEA安装阿里的代码规范插件
-
在保证需求的前提下，尽量控制技术债务的规模
-
后台Java代码规范遵照阿里Java代码规范执行
-
前台JS暂时规范参考Google规范执行
发起PR规范
-
每个 Pull Request 只做一件事
-
避免超大的 PR
-
记录下自己提交的原因
-
提交前，执行阿里代码规范插件的规范性检查，解决提示的全部
-
提交前，执行代码格式化保证风格的一致
-
有单元测试编写能力的人员，单元测试应覆盖全部逻辑，测试覆盖率应超过70%
-
保证代码能够100%自测试，并且进行真正测试
-
较大规模的提交需要交由测试人员测试后确认没有问题再提交PR
-
提交后检查CI构建结果
-
尽量回复所有评论，尊重评论者对 PR 的关心
PR评审规范
-
尊重PR提交者，评论和意见都要是建议性的而不是批判性的
-
如果意见不太确定建议与提交人员
-
检查代码实现与需求是否一致
-
检查代码实现与接口设计或架构设计是否保持一致
-
检查代码中方法，变量，类的命名是否规范
-
类采用名词，大驼峰命名，比如EmailProcesser
-
方法采用动词，小驼峰命名，比如validEmail
-
变量采用名词，小驼峰命名
-
静态变量采用大写下划线的模式
-
-
检查是否有明显安全漏洞，比如明文的密码，未加密的敏感信息
-
检查代码或设计中是否有对操作系统或环境的依赖，特别是考察是否有对于windows的依赖，原则上是不容许有这样的依赖，如无法避免需要技术负责人确认
-
综合考虑项目需求，进度和技术，世界没有完美的，需求必然出现变化，技术债务做到可控即可
