alt Hacker News> This allows reviewers to step through your pull request in small bites, with each set of changes scoped to a single aspect of the feature.
Is that a frequent way of reviewing? On GitHub you get shown all changes together in the review tab. You can select individual commits for closer inspection, but where is the benefit?
Replies
Balinares • today at 10:47 AM
A series of piecemeal self-contained changes is much easier to wrap your brain around comprehensively enough to detect logic issues.
I started doing exactly this and it's been invaluable.
➕ show 1 reply
steveklabnik • today at 11:19 AM
See this gist (and the discussion) https://news.ycombinator.com/item?id=41505266
jeltz • today at 11:17 AM
Yes, it us common among people who use git and it makes reviews of complex features much easier.
I suspect there is some weird habit that some people even like to overengineer their git history. Maybe it improves the pixel fame ratio or something.
For me it's satire. There are reasons for varying effort in creating PRs or patches, but attempts like this never seem to reason about reality. If I have to review, I want to see the code, not a clever story hidden in the commit history.