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

相关推荐
叫致寒吧10 小时前
Tomcat详解
java·tomcat
S***267514 小时前
基于SpringBoot和Leaflet的行政区划地图掩膜效果实战
java·spring boot·后端
马剑威(威哥爱编程)15 小时前
鸿蒙6开发视频播放器的屏幕方向适配问题
java·音视频·harmonyos
JIngJaneIL15 小时前
社区互助|社区交易|基于springboot+vue的社区互助交易系统(源码+数据库+文档)
java·数据库·vue.js·spring boot·论文·毕设·社区互助
V***u45315 小时前
MS SQL Server partition by 函数实战二 编排考场人员
java·服务器·开发语言
这是程序猿15 小时前
基于java的ssm框架旅游在线平台
java·开发语言·spring boot·spring·旅游·旅游在线平台
i***t91916 小时前
基于SpringBoot和PostGIS的云南与缅甸的千里边境线实战
java·spring boot·spring
k***082916 小时前
【监控】spring actuator源码速读
java·spring boot·spring
麦麦鸡腿堡16 小时前
Java_网络编程_InetAddress类与Socket类
java·服务器·网络
vx_dmxq21116 小时前
【PHP考研互助系统】(免费领源码+演示录像)|可做计算机毕设Java、Python、PHP、小程序APP、C#、爬虫大数据、单片机、文案
java·spring boot·mysql·考研·微信小程序·小程序·php