• @Schal330
    link
    491 month ago

    Removed double quotes, added single quotes

    • @iAvicenna
      link
      351 month ago

      someone later on -m “removed single quotes, added double quotes”

        • @[email protected]
          link
          fedilink
          English
          71 month ago

          Real talk: I have worked at a company where an exec briefly had the bright idea to rank developers on number of commits made.

          You can imagine how that turned out.

          He wasn’t an exec for very long.

          • @[email protected]
            link
            fedilink
            English
            31 month ago

            lmaoo, I can’t believe someone actually did that and was serious about it.
            I’m already imagining commits piling up with comments like:

            • “add a newline”
            • “move function up”
            • “rename variable users to user_list
            • “Revert previous commit”
            • “no John, it’s user_list and that’s it, don’t you dare change it again”

            He wasn’t an exec for very long.

            Good riddance I guess

            • @[email protected]
              link
              fedilink
              English
              51 month ago

              Oh, no, we can do better.

              Every single line of every single PR is an individual commit. And never squash-commit.

                • @[email protected]
                  link
                  fedilink
                  English
                  2
                  edit-2
                  1 month ago

                  If you really want to nuke it from orbit: do it for every character.

                  It is absolutely technically feasible to go further than that, but I think that’s the point of diminishing returns in terms of effort.

    • AwkwardLookMonkeyPuppet
      link
      English
      31 month ago

      You’re going to regret not adding the second sentence to the commit message some day.

      • @[email protected]
        link
        fedilink
        51 month ago

        Bold of you to assume there is a cohesive theme to dotfile commits. Usually it’s just been long enough that I don’t want to lose my latest tweaks

    • The Assman
      link
      fedilink
      81 month ago

      My commit messages have gotten extremely lazy since I start squashing all my commits down to one. I just describe the PR on the first commit message and write nonsense in all the others.

      • @[email protected]
        link
        fedilink
        51 month ago

        I’ve been adding the commit shas + messages to the final description, if anyone wants to see my exploratory work.

        • @[email protected]
          link
          fedilink
          21 month ago

          I know that if you are on the local repository where the commits were originally created they’ll remain accessible through recovery methods but AFAIK orphaned commits aren’t synced to other machines.

          • @[email protected]
            link
            fedilink
            2
            edit-2
            1 month ago

            That’s correct. This is for work, which uses GitHub. The dangling commits remain accessible via their sha through the web ui, so I can link them in the PR description. I don’t put them in the actual commit message.

            I think these are garbage collected eventually, but no idea on cadence. It’s long, anyway.

  • @alyth
    link
    271 month ago

    Tests: Add a few test files.

    • @eskimofry
      link
      161 month ago

      I thought we lost you … Jia Tan!

  • @camr_on
    link
    171 month ago

    God read my code and it made Him cry

  • Ving Thor
    link
    fedilink
    161 month ago

    Update coffee list and tex files.

    …and yes we have our office coffee list in a git repo.

    • @[email protected]
      link
      fedilink
      41 month ago

      Do new hires have to submit a pull request to get in on it? If so, I’m guessing the non-technical staff don’t get to drink a lot of coffee.

    • @[email protected]
      link
      fedilink
      English
      31 month ago

      Do you have any plans to open source your coffee list? I want to add this feature where I get an iced coffee in the morning

    • AwkwardLookMonkeyPuppet
      link
      English
      121 month ago

      “circumvented GDPR despite calling out strong ethical objections, because legal & management are forcing me to”

  • @[email protected]
    link
    fedilink
    131 month ago

    “28496 - there, it’s fucking fixed you twat waffle.”

    Ticketed bug bosses son found. Dude nagged his dad who nagged us until it got fixed. Boss doesn’t review code. And for the sake of a half dozen coworkers, I hope he never does.

  • @Sylvartas
    link
    12
    edit-2
    1 month ago

    “;”

    Previous commit was some stupid easy fix I didn’t even bother compiling. Well, I should have, because it was the first time in recent memory I committed some code with a missing semicolon…

    Before anyone asks : no, we don’t do reviews ¯\_(ツ)_/¯

  • JoshCodes
    link
    fedilink
    111 month ago

    git commit -m “if this doesn’t fix it I’m looking up availabilities at my nearest maccas”