[masklinn]

You know that you can edit the history in order to fold the small fixes and mistakes back into the commit they belong to right? It’s not an all or nothing situation.

> If someone wants to dig into a feature commit by commit, they can typically check the PR out and do it that way, but in practice, I've rarely seen that happen

Of course not, it’s a pain in the ass having to go back to the PR and trawl through a commits sequence you describe as essentially broken.

[afterburner5]

I know that, but what advantage do you get from trawling through a commit sequence, over the general feature with proper documentation? If you can't understand how the overall feature was implemented, that PR generally speaking is lacking enough context to be reviewed, so why did it get merged in?