英文阅读-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下能够满意。

相关推荐
Aevget1 天前
「Java EE开发指南」用MyEclipse开发的EJB开发工具(二)
java·ide·java-ee·eclipse·myeclipse
黄昏晓x1 天前
C++----多态
java·jvm·c++
Brookty1 天前
【算法】前缀和
java·学习·算法·前缀和·动态规划
少许极端1 天前
算法奇妙屋(七)-字符串操作
java·开发语言·数据结构·算法·字符串操作
懒羊羊不懒@1 天前
Java基础语法—字面量、变量详解、存储数据原理
java·开发语言
望获linux1 天前
【实时Linux实战系列】实时 Linux 的自动化基准测试框架
java·大数据·linux·运维·网络·elasticsearch·搜索引擎
Code blocks1 天前
GB28181视频服务wvp部署(一)
java·spring boot·后端
我命由我123451 天前
Spring Boot - Spring Boot 静态资源延迟响应(使用拦截器、使用过滤器、使用 ResourceResolver)
java·spring boot·后端·spring·java-ee·intellij-idea·intellij idea
Xzh04231 天前
前后端学习的交界
java·ajax·maven·axios·测试
豆沙沙包?1 天前
2025年--Lc201- 378. 有序矩阵中第 K 小的元素(排序)--Java版
java·线性代数·矩阵