• @CodeMonkey
      link
      837 days ago

      Senior developer tip: squash the evidence.

      • @[email protected]
        link
        fedilink
        286 days ago

        Principle developer tip: rewrite history to make yourself seem smarter.

        Soft reset the whole branch and commit a series of atomic and semantic patches (eg separating code, test, and refactor changes) that tell a clean narrative of the changeset to reviewers, future blamers.

        • JackbyDev
          link
          English
          36 days ago

          And then my team squashes those commits 😩

          • @[email protected]
            link
            fedilink
            16 days ago

            Yeah it sucks. If the commits are really helpful, you can just paste the git log into the PR/MR/CR body after it’s been merged

        • Elise
          link
          fedilink
          26 days ago

          Do you put effort into your commit messages before the rewrite, or just write something quick for yourself and then put in the effort later?

          • @[email protected]
            link
            fedilink
            26 days ago

            Depends, but usually I will put in the effort up front and maybe tweak them in an in[eractive rebase, or just manually copy+paste.

            If they’re worth saving. Sometimes you have to kill your darlings though

      • @SatouKazuma
        link
        67 days ago

        Senior developer tip: Squash the evidence management

    • @RonSijm
      link
      96 days ago
      git reset head~9
      git add -A
      git commit -am 'Rebased lol'
      git push -f