logoalt Hacker News

madoglast Wednesday at 8:54 AM2 repliesview on HN

Something like Gerrit. Instead of carefully crafting a logical series of patches that are all well documented with commit messages, PRs are just garbage filled diff soup of "fix typo" commits. I hate it. It's hard to review and seems to be based on putting the least amount of effort into proposing changes to the code. See https://gist.github.com/thoughtpolice/9c45287550a56b2047c631...


Replies

Cthulhu_last Wednesday at 9:41 AM

That's down to culture and (self) discipline, not tools.

show 1 reply
sublimefirelast Wednesday at 9:08 AM

Well but this is controllable, i.e. it is people who choose to do this not the platform. Very much an internal design choice.

show 1 reply