双方互相学习,你好我也好
23年腾讯有个文章,说cr就是群里@一下,5秒后恢复done就结束,讲的是业务复杂性带来的代码不可维护
大家也可简简单单蒙混过关,我见过的大多数也都是这样,但是利于成长么?遇到认真CR的人,对于大家来说都是一件好事情
开源社群争吵也不过是正常现象,争吵不必太在意,也不必不在意,心平气和互相探讨一些即可
前端eslint
后端java 阿里巴巴插件
加上提交前检查,和提交代码后github自动代码扫描,通过工具解决,这里要减少人为处理
甚至可以找到一些AI插件,提前CR代码,提出问题和修改意见
减小改动范围(依赖、构建环境,非必要不更换)
有时候为了修复问题,一次升级好多不相关依赖,实际上没有必要
环境也是
没有明确要解决什么问题,盲目变更会带来位置风向
单词、方法名称语义要合理,直观符合常识