• Feyd@programming.dev
    link
    fedilink
    arrow-up
    2
    ·
    9 days ago

    Squashing is good because otherwise you have commits that are unreviewed and broken in your main branch which confounds git bisect, which is largely the only time most people even look at commit history

    • lad@programming.dev
      link
      fedilink
      English
      arrow-up
      2
      ·
      9 days ago

      Largely yes, but also having it squashed requires better commit messages and comments to show what was done for what reason. But yes, bisect is the only reason I’m fine with squashing

      • Feyd@programming.dev
        link
        fedilink
        arrow-up
        1
        ·
        9 days ago

        It seems for some reason you assume people have better discipline for commit messages made in the course of a branch that will be merged, but that’s absolutely not true as a general rule. Additionally, even if the squashed commit message is bad, it will at least correspond to a PR in the forge.