• Quantenteilchen@discuss.tchncs.de
    link
    fedilink
    arrow-up
    7
    ·
    3 months ago

    I never thought about it and instantly wanted to reply “wait why can’t you do that‽” but now that I thought about it, what would you want the history to look like in that case? A slightly weird rebase? A single commit which seemingly copy pasted the entire other branch with no relation to it left behind?

    • ranzispa@mander.xyz
      link
      fedilink
      arrow-up
      18
      ·
      3 months ago

      Sometimes you really don’t want to look over the commit history of your colleagues. As long as it’s a small feature, a single commit is a pretty good option.

      Rather than:

      • implemented X
      • forgot this
      • oh, this was not needed
      • now tests actually pass
      • oops
      • fixed this
      • should be ready
      • Elvith Ma'for@feddit.org
        link
        fedilink
        arrow-up
        10
        ·
        3 months ago

        That’s basically my commit history for every repo where I need the pipeline to run to see if everything works.

        • ranzispa@mander.xyz
          link
          fedilink
          arrow-up
          1
          ·
          3 months ago

          When I do that I always have a Dev branch that I use as the production branch to run the actual calculations.

          When I get something working I merge it off, clean up the history a little bit, rebase main onto it and then rebase de onto main.

      • expr@piefed.social
        link
        fedilink
        English
        arrow-up
        5
        ·
        3 months ago

        It’s fine if the changes belong in a single commit. Otherwise, an interactive rebase to craft a clean, quality history before merging is much, much better.

    • mcv@lemmy.zip
      link
      fedilink
      arrow-up
      7
      ·
      3 months ago

      I’m not a fan of changing history in general. Rebase can also he dangerous.

      I think ultimately it’s a matter of scale. Sometimes it can be useful to look into the details of the development of a single feature, but in a large project, that rarely happens. I’m not a fan of squashing, but for large projects, it helps to keep your history manageable.

      • expr@piefed.social
        link
        fedilink
        English
        arrow-up
        4
        ·
        3 months ago

        Rebasing is not dangerous. You can always go back if something is not to your liking.

        You don’t rebase shared history, you use rebases to craft a clean, quality commit history for your own branches before merging. If everyone does this, then squashing is unnecessary, because garbage commits don’t exist. It is the far superior way of doing things if you actually care about having good commits.

        Keeping a quality history rather than squashing also makes many other git tools much better, such as git blame, git revert, git bisect, and so on.

        • mcv@lemmy.zip
          link
          fedilink
          arrow-up
          0
          ·
          3 months ago

          Rebasing is dangerous if you rebase shared history. If you rebase a local branch, you have to be aware of how much of that local branch you may already have shared.

          On top of that, if you’ve got a lot of commits you’re rebasing in a merge conflict that can become extremely repetitive.

          So ideally, you only rebase single commits that you haven’t pushed yet. As long as you do that: always pull main and rebase on top of that before you push single commits, rebasing is fine. But the more you deviate from that, the riskier it becomes.

          • Ethan@programming.dev
            link
            fedilink
            English
            arrow-up
            1
            ·
            3 months ago

            I constantly rebase my feature branches regardless of how many commits there are and whether I’ve pushed any of them. So long as no one else has checked out my branch it’s perfectly safe. Personally I find rebase merge conflicts far easier to work with. Traditional merge conflicts are “Here’s someone else’s changes, figure out how to merge them into your feature branch.” Rebase merge conflicts are “The main branch has changed since you made your changes. Re-apply your changes to the new base.” For me/my brain, the latter is so much easier. The only time I ever run into problems is when there are merges in the history I’m rebasing. Which I avoid by never merging into my feature branches, only rebasing.

            And if it goes wrong, just git rebase —abort. Or if you already completed the rebase, git reset —hard origin/YOUR-BRANCH. Or if you majorly fucked up, use git reflog to find a good commit and reset to that. Zero risk if you know what you’re doing.

          • expr@piefed.social
            link
            fedilink
            English
            arrow-up
            1
            ·
            3 months ago

            You don’t share feature branches. So you always know precisely what is shared history: the commit you branched from.

            The workflow is branch from shared history, rebase your branch as many times as necessary during development to craft a quality history, then merge back.

            I rebase dozens of times a day and have never had a single issue with it.

            If you’re bothered by repetitive merge conflicts (which, in my experience, are quite rare if you’re doing things correctly), that’s what git rerere is for.

            Rebasing is for crafting a quality history of your own commits (or getting your branch up to date with the trunk). Merging is for integrating your commits with the shared history.

              • expr@piefed.social
                link
                fedilink
                English
                arrow-up
                1
                ·
                3 months ago

                And I"m saying that doing with merges (and squashing) what should be done with rebases is bad. You can do it that way, but you shouldn’t, because it makes for worse history and less usable git tools.

    • psycotica0@lemmy.ca
      link
      fedilink
      arrow-up
      5
      ·
      edit-2
      3 months ago

      Yeah, I’m with you. I mean, git isn’t magic. You “can” squash anything, including a merge commit, by just being at the end result, running git reset <commit you want to be squashed off of> and then running a manual git add and commit there. That’s basically all a squash is.

      But what you’ll be left with us a single commit that contains all of the code from the branch you’re squashing and also all the code pulled in from every branch you merged, all written as though it all came from this one commit. And maybe that’s what you want? But it feels like also maybe it’s not?

    • kevincox@lemmy.ml
      link
      fedilink
      arrow-up
      2
      ·
      3 months ago

      I think it doesn’t really make sense. Because you can’t “squash” one commit. squash is taking multiple commits and making them one.

      When you do a “squash merge” you are really saying “squash all the commits that are on this branch and not the target” then merge.

      So you can’t “squash a merge commit” you need at least one additional commit to squash in.