英文阅读-LinkedIn‘s Tips for Highly Effective Code Review

LinkedIn的CR技巧

LinkedIn团队CodeReview经验与方法,原文来自https://thenewstack.io/linkedin-code-review/

总结

  • Do I Understand the "Why"?

    在提交pr的同时需要描述本次修改的"动机",有助于提高代码文档质量。

  • Am I Giving Positive Feedback?

    当reviewer看到优秀代码需要给出正反馈。

  • Is My Code Review Comment Explained Well?

    comment需要简洁易懂,比如"reduces duplication", "improves coverage"等等

  • Do I Appreciate the Submitter's Effort?

    每一次pr都需要被感谢,不管结果如何,使用谢谢

  • Would This Review Comment Be Useful to Me?

    减少不必要的comment,比如代码格式有问题,开发者需要将CR意见当成有用的工具

  • Is the "Testing Done" Section Thorough Enough?

    每一次变更都需要通过单元/接口测试

  • Am I Too Pedantic in My Review?

    养成CR习惯,不要当成一种负担。养成一种心态:别人要CR我的代码,至少我的代码在自己CR下能够满意。

相关推荐
2601_9620715714 小时前
【Java报错已解决】org.springframework.beans.factory.BeanCreationException
java·开发语言
zww894911115 小时前
酒馆预约系统开发实战:从需求分析到上线全流程指南
java·eclipse
zww894911118 小时前
匿名树洞系统开发实战:从需求分析到部署指南
java·eclipse
黑马程序员毕设19 小时前
基于Java的医院药品管理系统的优化设计与实现
java·开发语言·spring boot·小程序·架构·课程设计·毕设
木井巳19 小时前
【BFS/DFS 解决 FloodFill 算法】太平洋大西洋水流问题
java·算法·leetcode·深度优先·广度优先·宽度优先·推荐算法
白山编程大哥19 小时前
Java 集合算法:从排序、查找到底层原理的实战指南
java·python·算法
devpotato20 小时前
缓存与数据库更新顺序不一致问题
java·数据库·redis
xcl092520 小时前
酒馆预约系统开发实战:从需求分析到上线全流程指南
java·spring boot·需求分析
Kyrie_kk20 小时前
Java--IO--Path文件访问
java·后端
qinqinzqq20 小时前
Maven POM 格式、Schema 与 XSD:一篇讲透
java·maven