• catastrophicblues@lemmy.ca
    link
    fedilink
    arrow-up
    7
    ·
    11 months ago

    Oh god I feel so called out. I wish I paid more attention to my commit messages but I’m usually too busy fixing the directory structure and refactoring. Sigh.

    • brlemworld@lemmy.world
      link
      fedilink
      arrow-up
      2
      ·
      11 months ago

      My company collapses into a single commit at merge so idgaf what the commit message is anymore. Though I would prefer not collapsing them.

      • DoomBot5@lemmy.world
        link
        fedilink
        English
        arrow-up
        3
        arrow-down
        1
        ·
        11 months ago

        Master should just have the feature description commits, not the hundred commits it took to get there after refactoring the code for the 3rd time and pulling changes from master since it’s taken so long to get done.

      • Gyoza Power@discuss.tchncs.de
        link
        fedilink
        arrow-up
        1
        ·
        11 months ago

        I prefer that approach. We work with smaller tasks, so it makea more sense, plus it helps keep the master clean and if you want a more detailed view of the specific commits, you just have to click on the link to the PR. It’s a better way to organise it IMO

      • catastrophicblues@lemmy.ca
        link
        fedilink
        arrow-up
        1
        ·
        11 months ago

        Yeah I worked at a place like that, but it made sense because we were also expected to keep PRs small, so a good commit message for several squashed ones was perfectly fine.