Re:Show HN I made a git rebase TUI editor
by skydhash on Monday October 14, 2024 @12:59PM
(#41839413)
I treat PRs like the email workflow. You send a diff my way for a particular changes, I either accept it or reject it. Or I suggest modifications. Recursively. It’s the whole patch I’m interested in, not the implementation history (I’m not your tutor). Once approved, I make a commit in main for this diff.
[ Parent
| on HN ]
Re:Show HN I made a git rebase TUI editor
by keybored on Monday October 14, 2024 @02:35PM
(#41840461)
The classic email workflow is either one patch or a patch series. Where each patch becomes a commit. And each patch can be reviewed in isolation (like a commit).
It is not anemic like the squash workflow.
Re:Show HN I made a git rebase TUI editor
by skydhash on Monday October 14, 2024 @02:52PM
(#41840625)
There’s nothing stopping anyone from creating a PR series. My reasoning for the squash workflow is described here[0]. I just equate a PR to a patch. And it becomes a commit in the main branch. I don’t really care about the commits in the PR, just like no one cares about the commits that produced the patch in the email workflow.
Re:Show HN I made a git rebase TUI editor
by keybored on Monday October 14, 2024 @03:02PM
(#41840722)
> I don’t really care about the commits in the PR, just like no one cares about the commits that produced the patch in the email workflow.
They do care. They go through the trouble of reviewing it so that the resulting commit[1] that lands in the upstream repository is good.
[1] Presumably you don’t mean “they don’t care about the commit that produced the patch”… since the patch is just a transport format for the commit.
Re:Show HN I made a git rebase TUI editor
by skydhash on Monday October 14, 2024 @03:26PM
(#41840966)
> Presumably you don’t mean “they don’t care about the commit that produced the patch”… since the patch is just a transport format for the commit.
Commits. Not everyone will care to clean up their local history just to produce a single patch. You can git diff it out.
EDIT:
I was using "patch" for diff so scratch the above comment. Even then, when using a forge, I'd rather use squash unless everyone clean up their commit history when producing a PR.