PR 闲置时间太长？审查 PR 与创建 PR 同样重要-低调大师

PR 闲置时间太长？审查 PR 与创建 PR 同样重要

2022-01-01 495

软件交付智能平台 LinearB 的数据科学团队研究了来自 2.6 万名开发者的 73.3 万个 PR 和 390 万条评论，发现：

50% 的 PR 在其生命周期的 50.4% 的时间里处于闲置状态；
33% 的 PR 在其生命周期中闲置了 77.8%（高达）的时间；
参与调查的开发人员的平均周期时间为 6 天 + 5 小时；
这些开发人员的平均 PR 审查时间为 4 天 + 7 小时。

对此，LinearB 的 COO 兼联合创始人 Dan Lines 则认为，PR 过程中的闲置时间就是开发者流程中的一个 killer。并表示，PR 悖论给自己和团队带来了很大的困扰。根据解释，所谓 PR 悖论（Pull Request Paradox）就是：我刚刚写了一些代码，可以对我们的客户产生积极的影响，我有动力尽快发布它。我需要你的帮助，但你却很忙，而且有动力继续写你自己的代码。这种冲突就是 PR 悖论。

Dan 称，研究所得的数据意味着：

每块工作平均有两天的闲置时间，造成了生产力浪费。
此举损害了开发团队合并和发布代码的能力，从而阻止价值交付。
闲置的时间会导致情景意识的降低、代码质量的降低和努力的浪费。“在我们打开一个新的 PR 之后，每过一个小时，重新审视我们的代码的认知负荷就会增加。如果我在一两天后收到问题或修改请求，就很难再回到我原来的流程状态。闲置时间损害了质量，因为当生产中出现我几天或几周前写的代码的问题时，就更难调试了。”
导致团队承诺无法兑现。

不过 Dan 的观点也遭到了一些网友的反驳，以下是 Reddit 上网友讨论中的一些高赞回答：

IceSentry：

文章中提到，在任务之间很难找到时间进行 quick review。这就是你的问题所在。审查 PR 应该是一项与创建 PR 同样重要的任务。如果有公开的 PR，就不要开始工作。开始在一个新问题上工作只会使一切变得更糟，并使你的团队更加缓慢。

显然，要鼓励提交小的 PR；如果是大的 PR，则至少要尽量把它分成明确的提交。

grauenwolf：

审查 PR 应该是一项与创建 PR 同样重要的任务。

大众需要理解这一点。高质量的代码并不 cheap，尤其是在为项目定义设计模式的初期。

如果你在这一步上偷懒，你的代码将很快变得杂乱无章（big ball of mud）。

8BitSk8r：

请告诉我的同事这个。他们认为“agile”意味着“we don't plan we just react”。

此外，Dan 还在文章中列举了一些 PR 的替代方案。首先是"真正的"持续集成。现在很流行的一种说法是，CI 和 PR 是相互排斥的。基于主干的开发允许开发人员直接提交到主分支，而不需要任何形式的审查或合并过程。但 Dan 认为，这种方法可能只适用于最精英的团队；对 95% 的人来说，它的缺点要大于优点。

其次是有人提出了一种 Ship / Show / Ask 策略：这是一种分支策略，它结合了 PR 的功能和保持发送变更的能力。Changes 被归类为"Ship"（不经审查合并到主线）、"Show"（打开 PR 进行审查，但立即合并到主线）或 "Ask"（在合并之前打开 PR 进行讨论）。总的来说，此策略对于低风险的工作，选择直接合并而不审查或事后审查是有一定道理的。但是 Dan 指出它也存在一个问题，即目前大多数团队都没有适当的定义、流程和自动化来使其发挥作用。

此外还有结对编程（Pair programming），不过这一方法好像也不尽适用。“我知道很多开发团队使用结对来补充异步拉取请求审查，而我个人从未在一个使用结对来取代 PR 的团队工作过。”

Dan 表示，他不认为 PR 会消失。“根据我的经验，在合并之前让队友审查你的代码是提高质量和减少错误的最好、最实惠的方法。PR 在捕捉可维护性错误方面特别有效，而这些错误是很难通过自动测试发现的”。且很多开发人员都同意 PR 是提高学习和教学质量的好工具。

而 LinearB 团队也针对 PR 悖论推出了一个相关的提升 PR 审查效率的方案，并表示已收获了积极地反馈。详情可查看文章。

微信关注我们

原文链接：https://www.oschina.net/news/176589/pull-request-review

转载内容版权归作者及来源网站所有！

低调大师中文资讯倾力打造互联网数据资讯、行业资源、电子商务、移动互联网、网络营销平台。持续更新报道IT业界、互联网、市场资讯、驱动更新,是最及时权威的产业资讯及硬件资讯报道平台。

Piping —— 轻量级文件传输工具

Piping 是轻量级的开源文件传输工具，支持自托管，支持使用 curl、wget 下载，支持更广泛地在无浏览器的设备上使用，也可以用于不同设备间传输文本或者文件，或者在不同用户间传递资料等场景。 Piping 的传输方式基于 HTTP/HTTPS，使用 Stream 流式传输，可传输任何数据，比如屏幕共享、远程桌面、共享绘画、文字聊天等内容，也无大小限制。据称开发者曾测试不间断用 64 天传输了 1PB 文件。 Piping 还提供了美观的 Web UI 界面，打开即用：https://piping-ui.org/

2022-01-01

823

在云原生浪潮的推动下，容器和 Kubernetes 技术和应用发展迅猛。最近，云原生计算基金会 CNCF 发布了与 SlashData 联手撰写的最新版《云原生开发现状报告》，该报告显示，“Kubernetes 在过去的 12 个月取得了令人瞩目的增长——今天，全球共有 560 万开发人员在使用 Kubernetes。对于那些拥有 500 多名员工的大型组织而言，Kubernetes 和容器的采用率猛增，这意味着 Kubernetes 已经完全满足企业的需求。很多时候，开发人员甚至在没有意识到 Kubernetes 的情况下就在使用它。” 云原生已经成为数字经济技术的创新基石，与此同时，容器正在成为企业使用云的新界面，为企业的计算基础设施、应用架构、组织文化和研发流程带来新一轮变革。最近，阿里云容器服务团队几位核心成员在一次访谈中，分享了他们对企业级容器应用市场的变化和 2022 年容器技术重点发展方向的趋势观察。本文希望结合 2021 年阿里云云原生容器服务的重点事件，帮助大家梳理云原生容器领域的核心发展脉络，把握技术趋势走向。盘点：2021 阿里云云原生容器服务大事记 1. 全...

2022-01-01

678

资源下载

更多资源

优质分享App

近一个月的开发和优化，本站点的第一个app全新上线。该app采用极致压缩，本体才4.36MB。系统里面做了大量数据访问、缓存优化。方便用户在手机上查看文章。后续会推出HarmonyOS的适配版本。

Mario

马里奥是站在游戏界顶峰的超人气多面角色。马里奥靠吃蘑菇成长，特征是大鼻子、头戴帽子、身穿背带裤，还留着胡子。与他的双胞胎兄弟路易基一起，长年担任任天堂的招牌角色。

腾讯云软件源

为解决软件依赖安装时官方源访问速度慢的问题，腾讯云为一些软件搭建了缓存服务。您可以通过使用腾讯云软件源站来提升依赖包的安装速度。为了方便用户自由搭建服务架构，目前腾讯云软件源站支持公网访问和内网访问。

Rocky Linux

Rocky Linux（中文名：洛基）是由Gregory Kurtzer于2020年12月发起的企业级Linux发行版，作为CentOS稳定版停止维护后与RHEL（Red Hat Enterprise Linux）完全兼容的开源替代方案，由社区拥有并管理，支持x86_64、aarch64等架构。其通过重新编译RHEL源代码提供长期稳定性，采用模块化包装和SELinux安全架构，默认包含GNOME桌面环境及XFS文件系统，支持十年生命周期更新。