top | item 45371519 (no title) kaapipo | 5 months ago I mean, stacked PRs are a thing for a reason discuss order hn newest keriati1|5 months ago I can also recommend rather to use the stacked PR approach. We have it since years, PR review "issues" are not a thing for us.I still encourage do to a lot of small commits with good commit messages, but don't submit more then 2-3 or 4 commits in a single PR... epage|5 months ago I see stacked PRs as independent of this. PRs are a good unit of cohesion of changes, particularly changes that only make sense if later changes are also merged. kiitos|5 months ago stacked PRs are wildly reviewer-hostile, please do not do them
keriati1|5 months ago I can also recommend rather to use the stacked PR approach. We have it since years, PR review "issues" are not a thing for us.I still encourage do to a lot of small commits with good commit messages, but don't submit more then 2-3 or 4 commits in a single PR...
epage|5 months ago I see stacked PRs as independent of this. PRs are a good unit of cohesion of changes, particularly changes that only make sense if later changes are also merged.
keriati1|5 months ago
I still encourage do to a lot of small commits with good commit messages, but don't submit more then 2-3 or 4 commits in a single PR...
epage|5 months ago
kiitos|5 months ago