程序员提交 PR 的理想长度是多少?有人答:50 行代码!

💡 原文中文,约2600字,阅读约需7分钟。
📝

内容提要

本文讨论了Pull Request(PR)的理想长度为50行代码变更,根据数据分析,50行的PR审核与合并速度快40%,撤销可能性低15%,每行变更的审阅意见增加40%。PR的大小还与评论数、交付代码总量等因素有关。开发人员应以50行为目标,但也需要根据实际情况调整代码变更的大小。

🎯

关键要点

  • PR 的理想长度应为 50 行代码。
  • 50 行代码变更的审核与合并速度比 250 行快约 40%。
  • 50 行代码修改后被撤销的可能性低 15%。
  • 每行变更的审阅意见增加了 40%。
  • 推荐的 PR 大小范围为 25~100 行代码。
  • PR 越小,审查、合并和每行评论的时间越少。
  • PR 大小与审核与合并时间、撤销概率、平均评论数和年度代码变化总量有关。
  • 撤销率最低的 PR 代码修改量在 25~50 行之间。
  • PR 规模超过 1 万行后,撤销的安全性略有提升。
  • 随着 PR 规模的增长,评论的平均数逐渐减少。
  • 持续编写 20 行以下的 PR 将对编程速度产生重大影响。
  • PR 大小的中值为 40~80 行可最大化交付代码总量。
  • 开发人员应以 50 行为目标,并根据实际情况调整 PR 大小。
➡️

继续阅读