Well are we talking about commits pre- or post-merge? I don’t care how many commits you put into the PR / MR as long as they squash down to a single commit upon merge.
The reason people like to work this way is that it lets you very effectively respond to review. You address them by amending the commit to incorporate the feedback.
This enables good interdiff review, so you can re-review just the new stuff in the new version of the commit and not the entire thing all over again.
Comments
Well are we talking about commits pre- or post-merge? I don’t care how many commits you put into the PR / MR as long as they squash down to a single commit upon merge.
When you work this way, each commit is expected to be able to land independently.
Doesn’t sound like it leaves much room for error. How do you address PR / MR comments? Force push?
The reason people like to work this way is that it lets you very effectively respond to review. You address them by amending the commit to incorporate the feedback.
This enables good interdiff review, so you can re-review just the new stuff in the new version of the commit and not the entire thing all over again.