News for hackers. Stuff that matters.

A comment on Git-absorb: Git commit –fixup, but automatic. Parent comment

Re:Git-absorb: Git commit –fixup, but automatic by johnmaguire on Thursday September 26, 2024 @09:56AM (#41658445)
> If the problem is devs having to wait too long for PRs to be reviewed, breaking tasks up in to smaller diffs should exacerbate the problem, wouldn't it?

I find small diffs to be much easier to review and as a result I review them much practically as soon as they are posted. And I'm less likely to miss something.

[ Parent | on HN ]
Re:Git-absorb: Git commit –fixup, but automatic by dpkirchner on Thursday September 26, 2024 @11:30AM (#41659537)
For sure, I agree. However there is also a context switching cost and IMO switching between your IC work and, say, a dozen small diffs is rougher than a couple larger diffs. That opinion said I haven't experienced fast reviews myself, so maybe that's what I'm missing.
[ Parent | on HN ]