TE
TechEcho
Home24h TopNewestBestAskShowJobs
GitHubTwitter
Home

TechEcho

A tech news platform built with Next.js, providing global tech news and discussions.

GitHubTwitter

Home

HomeNewestBestAskShowJobs

Resources

HackerNews APIOriginal HackerNewsNext.js

© 2025 TechEcho. All rights reserved.

Show HN: Maiao, Stacked Diffs for GitHub

52 pointsby joaoqalvesover 2 years ago

8 comments

feifanover 2 years ago
&gt; Identify which of the files should be modified first.<p>I&#x27;m curious how many software engineers work this way — i.e. you need to make a big change, and you can determine a sequential order in which to make changes?<p>My working style is mostly jumping back-and-forth across all the relevant files and tests as I go, realizing new things that I need to change, and iterating until it all works end-to-end.<p>Am I the outlier, or is Maiao&#x27;s expected workflow the outlier?
评论 #33134873 未加载
评论 #33134313 未加载
评论 #33134864 未加载
atq2119over 2 years ago
Some high-level description is what this actually does would be useful.<p>The main problem with &quot;stacked diffs&quot; or &quot;dependent pull reviews&quot; on GitHub is that they don&#x27;t work with forks&#x2F;clones.<p>That is: you can push a branch to your personal clone and then submit a PR against the main repository&#x27;s development branch. That works fine.<p>You can push two stacked branches into the main repository and set up dependent pull requests (where pr1 targets the main development branch and pr2 targets pr1). That sort of works -- lots of things could be improved about the UI of that workflow, but it&#x27;s at least possible to do.<p>You <i>can&#x27;t</i> reasonably push two stacked branches to your personal clone and then set up dependent pull requests against the main development branch: pr2 that targets pr1 will simply not show up in the main repository.<p>Given the fact that the original use case of Git is to work with &quot;stacked diffs&quot;, it&#x27;s absolutely mind-blowing that GitHub still has no property sort for it.
joaoqalvesover 2 years ago
Relevant link:<p>* Stacked diffs vs Pull Requests [1]<p>1 - <a href="https:&#x2F;&#x2F;jg.gg&#x2F;2018&#x2F;09&#x2F;29&#x2F;stacked-diffs-versus-pull-requests&#x2F;" rel="nofollow">https:&#x2F;&#x2F;jg.gg&#x2F;2018&#x2F;09&#x2F;29&#x2F;stacked-diffs-versus-pull-requests&#x2F;</a>
评论 #33133399 未加载
fredrikaverpilover 2 years ago
This seems like a great tool. I’m personally hooked on the <a href="https:&#x2F;&#x2F;graphite.dev" rel="nofollow">https:&#x2F;&#x2F;graphite.dev</a> CLI and would just like to mention it for comparison.
bvrmnover 2 years ago
Or better: do not assign a task requiring 500 line commit. Reviewing stream of though consisting of 10 commits is not very productive as well.<p>Everybody wins if github&#x2F;gitlab starts to show diffs between `push -f` for a feature branch. Like in gerrit.
评论 #33133967 未加载
评论 #33133930 未加载
WirelessGigabitover 2 years ago
How does HN deal with refactorings that litterally touch 90% of the code, but just 3 lines?<p>Huge PR to refactor, and fatigue sets in. But we cannot PR chunks because the build cannot fail.
评论 #33133420 未加载
评论 #33133438 未加载
评论 #33133921 未加载
评论 #33135348 未加载
评论 #33133696 未加载
rockwotjover 2 years ago
I use spr and git fixup for this workflow and it&#x27;s really helped me breakup my changes better, managing multiple branches and stacking changes that way is a huge pain, but this workflow has been huge for me!<p>SPR: <a href="https:&#x2F;&#x2F;github.com&#x2F;getcord&#x2F;spr" rel="nofollow">https:&#x2F;&#x2F;github.com&#x2F;getcord&#x2F;spr</a><p>Git fixup: <a href="https:&#x2F;&#x2F;github.com&#x2F;keis&#x2F;git-fixup" rel="nofollow">https:&#x2F;&#x2F;github.com&#x2F;keis&#x2F;git-fixup</a>
nichocharover 2 years ago
Check out graphite. Been LOVING it. Team is awesome and product is evolving fast.