广东建的电商网站叫啥,网站备案 价格,制作网页的图,做网站过程视频作者#xff1a;Kief Morris
译者#xff1a;冬哥
原文#xff1a;https://infrastructure-as-code.com/book/2021/01/02/pull-requests.html
前 言 Github 引入了Pull Request拉取请求#xff08;简称PR#xff09;实践和相关的支持功能#xff0c;使运行开源项目的人…作者Kief Morris
译者冬哥
原文https://infrastructure-as-code.com/book/2021/01/02/pull-requests.html
前 言 Github 引入了Pull Request拉取请求简称PR实践和相关的支持功能使运行开源项目的人更容易接受来自他们信任的提交者群体之外的贡献。
Committer是被信任的主体可以定期更改代码库。但是需要评估来自随机外部人员的更改以确保其有效不会使项目朝着不需要的方向发展并且符合风格和质量标准。外部人员将他们提出的变更打包为拉取请求Committer可以轻松地将其作为一个单元进行审查和管理然后再将其合并到代码库中。 图 1拉取请求流程
尽管拉取请求旨在更容易地接受来自团队外部不受信任的人的贡献但现状是很多团队对内部人员使用拉取请求。这种做法已经变得如此普遍以至于许多人将其视为默认的“最佳”实践。有些人认为没有其他方法可以确保代码得到评审因为他们从未见过其他任何东西。
然而拉取请求会牺牲性能包括交付时间和质量。当管理来自未知人员的变更风险时这是一个值得做出的牺牲。外部人员可能不了解你项目的愿景和方向。他们在测试、代码质量和风格方面可能没有相同的习惯和规范。但是你自己团队成员内部应该共享这些规范。
针对自己团队成员的代码变更采用拉取请求就像让你的家人通过机场安全检查站进入你家一样这是针对不同问题的昂贵解决方案。
使用持续集成而不是拉取请求 软件交付过程应该针对流程和质量进行优化。将变更前置保持在较短的时间并在变更引入问题时提供快速反馈。这是支持持续集成(CI)的想法。CI 是在每个人的代码上不断合并和测试他们的做法。 图 2持续集成过程
“当他们工作时”是必不可少的。作为团队成员你不会等到完成某个功能或故事后才将代码集成到主干中。相反你经常 - 至少每天一次 - 将你的代码置于健康状态通过测试并将其与其他人的当前工作一起集成到主干中。
每次推送变更时CI 构建作业都会自动测试项目的主干。这意味着在投入太多时间之前你会立即发现你正在做的事情是否与另一个人正在做的事情发生冲突。当你认为已经完成了一个故事或功能却发现必须回去解决并重做这几天的努力会很糟糕。 图 3每次推送时都在集成代码上运行测试 拉取请求的麻烦
拉取请求会延迟集成。当完成工作认为已准备好与团队其他成员的工作进行集成时你创建一个拉取请求并等待有人对其进行审核。只有在其他人审查通过变更后才会将其与主干集成。
如果团队成员快速审查和集成拉取请求这仅比 CI 慢一点。也许他们会在你每次推送的30 分钟内回复并审核将你的代码变更与主干集成并针对它运行自动化测试。因此你可能会在 30-40 分钟左右之后发现与其他人的工作发生冲突。 图 4拉取请求与 CI 的反馈延迟
在实践中没有多少团队能在 30 分钟内可靠地解决拉取请求。在等待某人审查你的变更时你可以切换到另一个任务或开始处理新的修改。当发现存在问题时你需要切换回原始更改从而中断了当前的工作流程。
另一方面有效的 CI 构建应该在推送集成代码后的几分钟内完成测试 - 在我们的场景中最多 10 分钟。你几乎会立即发现该冲突因此可以对其进行调查和修复。
在从测试完全集成的代码中获得反馈之前你无需打断其他人的工作来要求他们对其进行审查。正如我将很快解释的那样你可能仍然需要有人审核变更。但是你可以利用更快的周期时间来提交、集成和测试自己的代码以便在要求他们审查之前进行多项更改。
即使团队中的每个人都很快地改变了拉取请求典型的做法是等到完成功能或故事的工作然后再将拉取请求与主干集成。大多数团队平均需要超过一天的时间来开发一个故事。因此典型的拉取请求流程不满足持续集成的最低要求即至少每天集成每个人的工作。
每天数次以编码、拉取、测试、推动和从集成测试中获取反馈的节奏工作会令人兴奋而拉取请求在节奏中引入人为延迟不可能产生这种兴奋感。
审查代码更改更好的方法
当 CI 与拉取请求的话题出现时不可避免地会有人站出来为拉取请求辩护以获取其他团队成员对更改的反馈。
必须有第二双眼睛如果不是更多查看代码更改。人类会发现测试没有发现的问题尤其是与可维护性和设计相关的问题。让人们审查彼此的代码还有助于团队在编码风格、编程习惯和质量要求等规范上趋于一致。在某些情况下例如受监管的环境需要由第二人审查每个更改。
然而最近拉取请求的流行似乎导致一些人认为没有其他方法可以用来审查代码变更。以下是你可以使用的一些实践而不会中断持续集成反馈周期。请记住完全可以根据需要进行多个实践的组合。 图 5配对以实现即时、持续的代码审查 结对编程没有任何形式的代码审查比结对编程更有效。反馈是即时的因此使用它进行改进的可能性要高得多。如果有人在你编写代码时告诉你有更好的方法那么你可以立即停止、学习并以更好的方式编写代码。如果有人在一天后告诉你你可以将其记录下来以备将来参考。但是要让你停止当前的工作并回去重做你已经完成的工作这会成为一个严重的问题。 定期审查如果合规性没有明确要求进行审查则可能不需要为每个代码更改设置一个门禁。你可能有定期的、预定的审查例如每周来检查自上次审查以来的代码更改。这作为小组练习尤其有效因为它创建了对话帮助人们学习和塑造团队的编码规范。 流水线审批如果你的团队使用持续交付流水线将变更交付到生产可以包括一个阶段要求有人授权变更进行。这在概念上类似于拉取请求因为它是交付过程中的一个门禁但当你将门禁放在代码集成和自动化测试之后。这样做意味着人们只花时间审查已经在技术上被证明是正确的代码。 图 6在集成和测试之后审查更改 结论 拉取请求与持续集成的不同之处在于在编写代码变更之后但在将其与主线集成之前需要人工审查代码更改。这会延迟从已完全集成代码的自动化测试中获取到反馈。
使用持续集成代码要么在编写时结对进行审查要么在集成和测试之后进行审查。针对集成和测试变更的循环进行优化意味着你可以更频繁地运行此循环。更频繁的编码和集成循环鼓励开发人员进行更小更频繁的提交从而提高质量和流程。
