6 ms·
Commits aren't always perfect. Sometimes I'll make the unit test first, which fails CI and the next set of commits implements the behavior.
by Forge36 2y ago
Commits aren't always perfect.
Sometimes I'll make the unit test first, which fails CI and the next set of commits implements the behavior.
- trashburger 2y agoBy doing this, you break commit atomicity and make bisects hell. Please don’t do this. Commits aren’t perfect at first for sure, but they should be by the time you make them reviewable.
- snatchpiesinger 2y agoIt's fine to break commit atomicity on feature branches. You can use git bisect --first-parent on you development/master branch.
- trashburger 2y agoI completely disagree. In doing so you lose all visibility into the components and gradual evolution of the code that atomic commits provide. Same thing with squashing (which is just the worst).