3 ms·
Yes. Much easier to roll back things, cherry pick things and blame people ;) Don't you dare merge a PR with 30 commits.
by beders 5y ago
Yes.
Much easier to roll back things, cherry pick things and blame people ;)
Don't you dare merge a PR with 30 commits.
- conradludgate 5y ago+1 to this. I like the mantra that every commit is deployable
- seba_dos1 5y ago"Every commit is deployable" isn't at odds with merging multi-commit MRs, provided that the submitter actually did their job well. I find squash-merging to be an ugly workaround, not a proper solution. Good non-trivial merge requests should usually consist of multiple commits that improve readability, revertability, cherry-pickability and blamability ;)