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

相关推荐
stein_java29 分钟前
springMVC-10验证及国际化
java·spring
weixin_4786897632 分钟前
C++ 对 C 的兼容性
java·c语言·c++
LUCIAZZZ1 小时前
HikariCP数据库连接池原理解析
java·jvm·数据库·spring·springboot·线程池·连接池
sky_ph1 小时前
JAVA-GC浅析(二)G1(Garbage First)回收器
java·后端
IDRSolutions_CN2 小时前
PDF 转 HTML5 —— HTML5 填充图形不支持 Even-Odd 奇偶规则?(第二部分)
java·经验分享·pdf·软件工程·团队开发
hello早上好2 小时前
Spring不同类型的ApplicationContext的创建方式
java·后端·架构
HelloWord~3 小时前
SpringSecurity+vue通用权限系统2
java·vue.js
让我上个超影吧3 小时前
黑马点评【基于redis实现共享session登录】
java·redis
BillKu4 小时前
Java + Spring Boot + Mybatis 插入数据后,获取自增 id 的方法
java·tomcat·mybatis
全栈凯哥4 小时前
Java详解LeetCode 热题 100(26):LeetCode 142. 环形链表 II(Linked List Cycle II)详解
java·算法·leetcode·链表