- 尊重他人，就事论事，对事不对人，毕竟每个人都写过烂代码；
- PR 中的每一个 commit log 都应该可以和代码对应，方便 review；
- 尽量不要发太大的 PR，以免引起 reviewer 的恐慌；
- 建议保证一个 PR 的粒度和专注，最好不要出现一个 PR 里又有重构又加新 feature 的情况，同样容易引起 reviewer 的恐慌；
- 提 PR 之前请确保在本地或测试环境一切正常；
- reviewee 如果接受 re
本文介绍了项目代码审查的规则，包括尊重他人、保持commit的清晰对应、避免大型PR、确保本地测试正常、及时反馈修改、遵循LGTM原则、避免自我合并、使用特定分支推送、考虑单元测试和安全性审查。同时，还提到了与上游仓库同步的正确方法，以及因错误导致问题的责任承担和团队建设活动。
最低0.47元/天 解锁文章
扫一扫
9151
被折叠的 条评论
为什么被折叠?
到【灌水乐园】发言
