态度

双方互相学习,你好我也好

23年腾讯有个文章,说cr就是群里@一下,5秒后恢复done就结束,讲的是业务复杂性带来的代码不可维护

大家也可简简单单蒙混过关,我见过的大多数也都是这样,但是利于成长么?遇到认真CR的人,对于大家来说都是一件好事情

开源社群争吵也不过是正常现象,争吵不必太在意,也不必不在意,心平气和互相探讨一些即可

格式规范

前端eslint

后端java 阿里巴巴插件

加上提交前检查,和提交代码后github自动代码扫描,通过工具解决,这里要减少人为处理

甚至可以找到一些AI插件,提前CR代码,提出问题和修改意见

减小改动范围

减小改动范围(依赖、构建环境,非必要不更换)

有时候为了修复问题,一次升级好多不相关依赖,实际上没有必要

环境也是

没有明确要解决什么问题,盲目变更会带来位置风向

语意合理

单词、方法名称语义要合理,直观符合常识

扩展性