• @grrgyle@slrpnk.net
    link
    fedilink
    285 months 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.

      • @grrgyle@slrpnk.net
        link
        fedilink
        15 months 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
      25 months 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?

      • @grrgyle@slrpnk.net
        link
        fedilink
        25 months 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