• xanu@lemmy.world
    link
    fedilink
    arrow-up
    5
    ·
    1 year ago

    To be fair, wasn’t the vim codebase entirely committed by a single person? He did that with everyone and, while I don’t agree with that at all, it reads less like elitism / stolen credit than this particular story.

    I may be wrong about that, so feel free to correct me 😊 either way, people should be credited for the work they do! and preferably not in the footnotes of a commit authored by someone else that didn’t fix the bug

    • JackbyDev
      link
      fedilink
      English
      arrow-up
      4
      ·
      1 year ago

      No, you’re correct, it definitely was all (or mostly) committed by Bram. That’s part of why I was saying it didn’t feel as bad but I didn’t think it was relevant to mention. But yes, you’re definitely correct.

      • aard@kyu.de
        link
        fedilink
        arrow-up
        1
        ·
        1 year ago

        Git has different fields for author and committer - and modifying a commit should leave the author field intact, and just change the committer field. It is possible that github does something weird (I’m usually not doing much in their web UI) - but coming from working with git directly I’d expect you to be present in the author field.

        • JackbyDev
          link
          fedilink
          English
          arrow-up
          2
          ·
          1 year ago

          I didn’t write the content of that commit. Author and committer being different is for things like rebasing commits written by other people.

          • aard@kyu.de
            link
            fedilink
            arrow-up
            2
            ·
            1 year ago

            You mentioned a pull request, and that it got edited - which in my workflow is pulling the commit and amending it.

            • JackbyDev
              link
              fedilink
              English
              arrow-up
              1
              ·
              1 year ago

              Okay, I probably misspoke about the technicalities. I opened a pull request, then they made a new commit and closed the PR (like it was an issue) and didn’t touch the commit. Hope that makes sense now.