草庐IT

git - Github 中的依赖 pull 请求是否可行？

coder 2023-06-23 原文

目前我正在处理一个非常大的 pull 请求。为了以某种方式保持代码审查的可管理性，我们的想法是将完整的 pull 请求拆分为独立的部分，但这些部分相互依赖。

一个例子是:

  • pull 请求 1:创建接口(interface):接口(interface) A 和 B 并重组代码
  • pull 请求 2:接口(interface) A 实现和测试(取决于 pull 请求 1)
  • pull 请求 3:接口(interface) B 实现和测试(取决于 pull 请求 2)
  • pull 请求 4:实现的混合测试(取决于 2 + 3)

Github 中有没有一种方法可以同时提交所有四个带有依赖项的 pull 请求？

最佳答案

2022 年更新:虽然 GitHub 仍然没有对依赖/堆叠 pull 请求的原生支持，但现在有一些工具可以更轻松地管理对 GitHub 友好的分支层次结构。我最喜欢的是ghstack ，它采用一堆提交并将每个提交变成它自己的分支，GitHub 可以在不丢失 PR 历史记录的情况下理解它。然后，您可以使用交互式 rebase 编辑、重新排序、拆分和删除提交，PR 将相应更新。

据我所知，这是不可能的，在我看来，与其他代码审查工具相比，这是 GitHub 的主要缺点之一。当您推送相互依赖的提交时，Gerrit 会自动设置依赖代码审查，而在 Phabricator 中，这更痛苦，但仍然有可能。

同样要记住，人们使用 GitHub PR 的方式有多种。正常的开源协作方式是 fork 一个存储库并提交一个跨存储库的 pull 请求，但在其他情况下(例如在一个组织内)，您可能会在同一存储库中提交所有差异的 pull 请求。我认为在单个存储库中，按照依赖 pull 请求的方式获得一些东西更合理，因为您可以在该存储库中设置提交/分支结构。

这是一篇描述如何获得依赖 pull 请求的一些优势的博客文章，我认为这需要所有提交都在同一个 repo 中: http://graysonkoonce.com/stacked-pull-requests-keeping-github-diffs-small/

总结:

  • 要提交 5 个相关的 pull 请求，请创建一个 5 级深的分支层次结构，并根据前一个分支将每个 PR 作为该分支提交。
  • 要更新评论 2(共 5 条)，将更新推送到分支 2，然后执行 3 次 merge 操作以将更改整合到评论 3、4 和 5 中。
  • 您需要一次完成所有更改，因为 GitHub 不支持更新 PR 的目标分支。在示例中，所有 5 次代码审查都作为单个提交登陆。GitHub 现在支持更新 PR 的基础分支，因此 PR 可以一次登陆一个。

这种方法似乎适用于最好以较小的部分进行审查的巨大更改(尽管与 git rebase -i 之类的东西相比，维护一个 n 级深的分支层次结构是一件痛苦的事情)，但它并没有真正允许“代码审查管道”，您可以在审查的不同阶段有依赖差异，并且可以在审查时更早地登陆。

其他一些似乎也指出了限制的互联网资源:

https://www.quora.com/Is-there-a-good-system-for-adding-multiple-pull-requests-to-GitHub

https://muffinresearch.co.uk/how-do-you-deal-with-dependent-branches-on-github/

我的理解是，使用 GitHub PR 的人通常只是尝试构建他们的工作流程，而不依赖于依赖代码审查。一些例子:

  • 与其将更改分解为可独立审查的增量步骤，不如将其作为一次全部完成的整体代码审查提交。 GitHub 仍然允许您将代码审查分成审查者可以查看的多个提交，但它们不能被独立批准/登陆。
  • 尝试构建您的工作，以便在代码的不相关部分进行更改，并将它们放在您可以用来提交独立 PR 的独立分支上。您仍然可以保留一个本地分支，并使用精选或其他方法应用所有更改。
  • 如果您的更改依赖于未完成的 PR，只需等待 PR 被接受并 merge ，然后再提交新的 PR。你可以在某个地方提到你有另一个 PR 依赖于那个 PR，也许这会激励代码审查者更早地到达那个 PR。

关于git - Github 中的依赖 pull 请求是否可行？，我们在Stack Overflow上找到一个类似的问题： https://stackoverflow.com/questions/26619478/

可行Githubpull储库stronggitpull-request

有关git - Github 中的依赖 pull 请求是否可行？的更多相关文章

  1. ruby - 如何从 ruby​​ 中的字符串运行任意对象方法？ - 2

    总的来说，我对ruby​​还比较陌生，我正在为我正在创建的对象编写一些rspec测试用例。许多测试用例都非常基础，我只是想确保正确填充和返回值。我想知道是否有办法使用循环结构来执行此操作。不必为我要测试的每个方法都设置一个assertEquals。例如:describeitem,"TestingtheItem"doit"willhaveanullvaluetostart"doitem=Item.new#HereIcoulddotheitem.name.shouldbe_nil#thenIcoulddoitem.category.shouldbe_nilendend但我想要一些方法来使用

  2. ruby - 其他文件中的 Rake 任务 - 2

    我试图在一个项目中使用rake，如果我把所有东西都放到Rakefile中，它会很大并且很难读取/找到东西，所以我试着将每个命名空间放在lib/rake中它自己的文件中，我添加了这个到我的rake文件的顶部:Dir['#{File.dirname(__FILE__)}/lib/rake/*.rake'].map{|f|requiref}它加载文件没问题，但没有任务。我现在只有一个.rake文件作为测试，名为“servers.rake”，它看起来像这样:namespace:serverdotask:testdoputs"test"endend所以当我运行rakeserver:testid时

  3. ruby-on-rails - Ruby net/ldap 模块中的内存泄漏 - 2

    作为我的Rails应用程序的一部分，我编写了一个小导入程序，它从我们的LDAP系统中吸取数据并将其塞入一个用户表中。不幸的是，与LDAP相关的代码在遍历我们的32K用户时泄漏了大量内存，我一直无法弄清楚如何解决这个问题。这个问题似乎在某种程度上与LDAP库有关，因为当我删除对LDAP内容的调用时，内存使用情况会很好地稳定下来。此外，不断增加的对象是Net::BER::BerIdentifiedString和Net::BER::BerIdentifiedArray，它们都是LDAP库的一部分。当我运行导入时，内存使用量最终达到超过1GB的峰值。如果问题存在，我需要找到一些方法来更正我的代

  4. ruby-on-rails - Rails 3 中的多个路由文件 - 2

    Rails2.3可以选择随时使用RouteSet#add_configuration_file添加更多路由。是否可以在Rails3项目中做同样的事情？ 最佳答案 在config/application.rb中:config.paths.config.routes在Rails3.2(也可能是Rails3.1)中，使用:config.paths["config/routes"] 关于ruby-on-rails-Rails3中的多个路由文件，我们在StackOverflow上找到一个类似的问题

  5. ruby-on-rails - 如何验证 update_all 是否实际在 Rails 中更新 - 2

    给定这段代码defcreate@upgrades=User.update_all(["role=?","upgraded"],:id=>params[:upgrade])redirect_toadmin_upgrades_path,:notice=>"Successfullyupgradeduser."end我如何在该操作中实际验证它们是否已保存或未重定向到适当的页面和消息？ 最佳答案 在Rails3中，update_all不返回任何有意义的信息，除了已更新的记录数(这可能取决于您的DBMS是否返回该信息)。http://ar.ru

  6. ruby-on-rails - Rails - 一个 View 中的多个模型 - 2

    我需要从一个View访问多个模型。以前，我的links_controller仅用于提供以不同方式排序的链接资源。现在我想包括一个部分(我假设)显示按分数排序的顶级用户(@users=User.all.sort_by(&:score))我知道我可以将此代码插入每个链接操作并从View访问它，但这似乎不是“ruby方式”，我将需要在不久的将来访问更多模型。这可能会变得很脏，是否有针对这种情况的任何技术？注意事项:我认为我的应用程序正朝着单一格式和动态页面内容的方向发展，本质上是一个典型的网络应用程序。我知道before_filter但考虑到我希望应用程序进入的方向，这似乎很麻烦。最终从任何

  7. ruby-on-rails - Rails 3.2.1 中 ActionMailer 中的未定义方法 'default_content_type=' - 2

    我在我的项目中添加了一个系统来重置用户密码并通过电子邮件将密码发送给他，以防他忘记密码。昨天它运行良好(当我实现它时)。当我今天尝试启动服务器时，出现以下错误。=>BootingWEBrick=>Rails3.2.1applicationstartingindevelopmentonhttp://0.0.0.0:3000=>Callwith-dtodetach=>Ctrl-CtoshutdownserverExiting/Users/vinayshenoy/.rvm/gems/ruby-1.9.3-p0/gems/actionmailer-3.2.1/lib/action_mailer

  8. ruby-on-rails - Rails 应用程序中的 Rails : How are you using application_controller. rb 是新手吗？ - 2

    刚入门rails，开始慢慢理解。有人可以解释或给我一些关于在application_controller中编码的好处或时间和原因的想法吗？有哪些用例。您如何为Rails应用程序使用应用程序Controller？我不想在那里放太多代码，因为据我了解，每个请求都会调用此Controller。这是真的？ 最佳答案 ApplicationController实际上是您应用程序中的每个其他Controller都将从中继承的类(尽管这不是强制性的)。我同意不要用太多代码弄乱它并保持干净整洁的态度，尽管在某些情况下ApplicationContr

  9. ruby-on-rails - form_for 中不在模型中的自定义字段 - 2

    我想向我的Controller传递一个参数，它是一个简单的复选框，但我不知道如何在模型的form_for中引入它，这是我的观点:{:id=>'go_finance'}do|f|%>Transferirde:para:Entrada:"input",:placeholder=>"Quantofoiganho?"%>Saída:"output",:placeholder=>"Quantofoigasto?"%>Nota:我想做一个额外的复选框，但我该怎么做，模型中没有一个对象，而是一个要检查的对象，以便在Controller中创建一个ifelse，如果没有检查，请帮助我，非常感谢,谢谢

  10. ruby - 检查数组是否在增加 - 2

    这个问题在这里已经有了答案:Checktoseeifanarrayisalreadysorted?(8个答案)关闭9年前。我只是想知道是否有办法检查数组是否在增加？这是我的解决方案，但我正在寻找更漂亮的方法:n=-1@arr.flatten.each{|e|returnfalseife

随机推荐