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

相关推荐
缓慢更新4 小时前
审批的加签与会签:多人意见怎么合
android·java·数据库·低代码·rxjava
yuniko-n5 小时前
【JUC】volatile 非原子性深入解析
java
sun_weitao5 小时前
极简版 MCP 工具实现方案
java·linux·服务器
金銀銅鐵6 小时前
[Java] 展示增删元素时 LinkedList 的内部变化
java·后端·ai编程
布吉岛的石头7 小时前
Java 程序员第 50 阶段3:数据工程:预训练语料清洗与去重
java·开发语言·人工智能·深度学习·transformer
大文说跨境7 小时前
防关联浏览器的指纹覆盖到什么程度:六类参数的实测与自检方法
java·指纹浏览器·tiktok运营
Carol06307 小时前
上肢及肩肘关节
java·服务器·数据库
yliucherish7 小时前
Log4Shell 复现实录:我读 OpenJDK 源码,发现两道防线的默认值竟然相反
java
维拉德8 小时前
Java新人第一次接打标需求:从抓包找接口到客户端展示
java
luobobo8968 小时前
支付回调验签连续失败三天,最后发现是网关的时间戳单位问题
java