| ▲ | dean_stratakos 4 hours ago | |
In our experience, it's difficult to create well-mannered commits as you code and new ideas pop into your head or you iterate on different designs (even for LLMs). One concept we toyed around with was telling an LLM to re-do a branch using "perfect commits" right before putting up a PR. But even then you might discover new edge cases and have to tack them on as additional commits. We thought git wasn't the right level of abstraction and decided to tackle things at the PR level instead. Curious to hear your experiences! | ||
| ▲ | tasuki 33 minutes ago | parent | next [-] | |
> In our experience, it's difficult to create well-mannered commits Sure, it is. But it's worth it, not just for code review, but for a myriad other things: bisect, blame, log, etc. Your tool makes one thing (the code review) easier, while decreasing people's motivation to make well-mannered commits, thus making everything else (bisect etc) worse. I'm sure it's net positive in some cases, and I think it's net negative in other cases. | ||
| ▲ | tasuki an hour ago | parent | prev [-] | |
> But even then you might discover new edge cases and have to tack them on as additional commits. Have you heard about `rebase -i` ? | ||