• Pup Biru@aussie.zone
    link
    fedilink
    English
    arrow-up
    1
    ·
    2 days ago

    Squashed commits are not atomic … overall task requires modifying multiple different systems

    that’s why monorepos exist

    i’d say squashed commits aren’t always atomic, but this is one of the biggest reasons people add the complexity of a monorepo: if changes cross multiple systems, ideally their merge/revert should be an atomic operation

    you either have deployment complexity (ensuring the feature is in all deployed systems before switching over), code complexity (dealing with the feature only maybe exiting in parts of the system), or repo complexity (where tools manage a monorepo and thus commits and PR/MRs are atomic across your system)

    • expr@piefed.social
      link
      fedilink
      English
      arrow-up
      1
      ·
      2 days ago

      Monorepos don’t really change anything. Squashed commits are still not atomic, unless the MR is small enough to fit into a single logical commit. Changes made to say, a database query are distinct from changes made to route handling, yet both might be needed for the overall feature. They don’t belong in the same commit in history.

      • Pup Biru@aussie.zone
        link
        fedilink
        English
        arrow-up
        1
        ·
        1 day ago

        hard disagree on what belongs in the same commit history… a single merge should be an entire feature, and your commit history should read like a change log