News for hackers. Stuff that matters.

A comment on Althttpd: Simple webserver in a single C file. Parent comment

Re:Althttpd: Simple webserver in a single C file by user-the-name on Friday June 11, 2021 @10:10AM (#27473571)
> And many tech leads simply don't have the time to review every single WIP commit

Do you usually review every commit? I usually just review the diff between the PR'd branch and master, as does everyone I work with.

[ Parent | on HN ]
Re:Althttpd: Simple webserver in a single C file by pdimitar on Friday June 11, 2021 @10:15AM (#27473652)
Exactly, and that's why we use squashed commits.
[ Parent | on HN ]
Re:Althttpd: Simple webserver in a single C file by user-the-name on Friday June 11, 2021 @11:33AM (#27474718)
Why? You don't need to. GitHub will show you that diff, git itself will show you that diff. There is no need to permanently rewrite history to do this. That makes no sense.
[ Parent | on HN ]
Re:Althttpd: Simple webserver in a single C file by pdimitar on Friday June 11, 2021 @11:38AM (#27474773)
For the last time: squashed commits help teams who need the main branch be a high-level history of delivered features and fixed bugs. Where one commit is one delivered feature or a bug fixed. That's the idea.

Whether you think that "makes no sense" is inconsequential. Many people find it very meaningful.

[ Parent | on HN ]