I believe that Ladybird has more funding and better support for the web, but Servo wins in performance. Though, they’re hard to compare directly!

  • bjorney@lemmy.ca
    link
    fedilink
    English
    arrow-up
    9
    arrow-down
    3
    ·
    5 days ago

    Yes, I’m sure that PR would have been accepted instead /s

    But you’re right, it doesn’t matter at all, the reasonable thing to do would have been for the guy to spend 3 seconds clicking the accept and merge button, or 6 seconds making your change. instead he wrote a comment stating that inclusive language has no place in his project

    • sugar_in_your_tea@sh.itjust.works
      link
      fedilink
      English
      arrow-up
      4
      arrow-down
      5
      ·
      5 days ago

      Here are the issues I see:

      1. That’s his only PR in this repo, and it just changed one word that wasn’t incorrect for one that’s preferable to the author
      2. The documentation in question is talking about an OS user, not a user of the software, so gender doesn’t apply (“it” would be totally appropriate)
      3. If goes against established norms here, it would be like changing a pronoun for a ship from “she” to “they”

      My suggestion sidesteps the issue entirely by avoiding pronouns, which doesn’t violate norms at all here.

      He didn’t say anything about inclusive language not being welcome, he said politically motivated changes aren’t welcome. If there’s documentation referencing users of telhe software, I’m guessing a change using inclusive language would be treated very differently.

      • bjorney@lemmy.ca
        link
        fedilink
        English
        arrow-up
        6
        arrow-down
        1
        ·
        5 days ago

        If goes against established norms here

        What’s the established norm here. All people compiling software by source are male?

        he said politically motivated changes aren’t welcome

        What’s politically motivated about changing “he” to “they”. As you said, gender doesn’t apply here, so the neutral word is literally preferable.

        • sugar_in_your_tea@sh.itjust.works
          link
          fedilink
          English
          arrow-up
          2
          arrow-down
          5
          ·
          5 days ago

          What’s the established norm here

          I’m honestly not sure, most OS projects I’ve seen use passive voice like I provided, because gender doesn’t make sense. I’ve seen a handful of projects select “he” for system users (e.g. root, nobody, etc), so that seems like the norm here, if there is one. Or it could be that the project uses “he” elsewhere to refer to these system users.

          Here’s the documentation:

          Note that the anon user is able to become root without password by default, as a development convenience. To prevent this, remove anon from the wheel group and <pronoun> will no longer be able to run /bin/su.

          This isn’t referring to an actual person, it’s referring to a system user created by the build script in the target operating system (SerenityOS). The user will never be used by an actual human, so any gender selected here is irrelevant, and there should be no preference for male, female, or a third gender.

          That’s why I prefer the passive voice because no gender makes sense, and it just looks weird.

          If I was the maintainer, I too would probably reject the PR because it didn’t remove the gender entirely. Most technical writing does that, because selecting a gender makes no sense.

          • bjorney@lemmy.ca
            link
            fedilink
            English
            arrow-up
            6
            arrow-down
            1
            ·
            5 days ago

            If I was the maintainer, I too would probably reject the PR because it didn’t remove the gender entirely.

            Cool, but that isn’t what happened here. The PR was closed immediately because the maintainer considered using gender neutral pronouns “personal politics” - he had ample opportunity to clarify his stance, or simply comment ‘resubmit in passive voice’, but he didn’t. Clearly the problem wasn’t the active voice, it was the summary of the change, because when that exact same PR was re-submitted much later with a commit message of ‘Fix some minor ESL grammar issues’, it was accepted with no discussion

            As an aside, I absolutely disagree with the use of passive voice. It’s more verbose, and harder for the reader to comprehend. It’s why every style guide (APA, Chicago, IEEE, etc) recommends sticking to active voice, especially in the context of ‘doing things’.

            • sugar_in_your_tea@sh.itjust.works
              link
              fedilink
              English
              arrow-up
              2
              arrow-down
              6
              ·
              edit-2
              5 days ago

              the maintainer considered using gender neutral pronouns “personal politics”

              In this case, yes. Context matters a lot here, and the context is that this didn’t refer to any human user, but a system user.

              What’s the gender of root? The question doesn’t make sense, because root isn’t alive, it’s a technical concept. What gender is your PC? A directory? It’s the same idea, it doesn’t make sense.

              However, switching it from one pronoun to another is politically motivated in the sense that it’s virtue signaling a certain brand of inclusiveness. The only gender that could make sense is whatever is used most frequently, e.g. w/ ships we use “she/her” for whatever reason, despite gender having absolutely no reason to exist. If a gender is used, I’ve seen the masculine, but again, that’s incredibly rare because any technical writer worth their salt would avoid the use of genders altogether because it doesn’t make sense to use a gender in that context.

              And why would he need to clarify anything? The simplest explanation is that this person is knee-jerk reacting to pronouns, and trying to change something that doesn’t matter at all because they see pronoun and think “must be gender neutral.” If they took a couple seconds to think, they would’ve realized that gender doesn’t matter at all here. It’s a useless change, and the reviewer shouldn’t spend any time on it at all.

              I’ve rejected tons of minor changes (e.g. whitespace changes) because it seemed the user was just looking to get their name in the commit log to build a resume or something. That’s a waste of everyone’s time, and this change looks no different.

              It’s why every style guide (APA, Chicago, IEEE, etc) recommends sticking to active voice, especially in the context of ‘doing things’.

              Yeah, that makes sense for most things, but for technical writing, the most important thing is clarity. Throwing a gender where it doesn’t belong is a distraction, and using active voice where it doesn’t belong is as well.

              Check out The Elements of Nonsexist Usage: A Guide to Inclusive Spoken and Written English, in Chapter 4 the author recommends exactly what I’ve outlined here: use “the user” instead of “he/she.” Here’s as Stack Exchange discussion about just that (which mentions this book):

              Replace gender-specific possessive pronouns with “the.” Instead of “When the user types his password…” try “When the user types the password.”

              If the PR removed gender entirely, it would’ve had a better change of being considered. It’s still a largely worthless PR though, unless it’s fixing something tangible in the documentation.

              • bjorney@lemmy.ca
                link
                fedilink
                English
                arrow-up
                6
                ·
                5 days ago

                You seem content to entirely gloss over the issue, which isn’t the pros/cons of a particular writing style, it’s that the maintainer could have said ANY of the things you said, but he didn’t

                • sugar_in_your_tea@sh.itjust.works
                  link
                  fedilink
                  English
                  arrow-up
                  1
                  arrow-down
                  5
                  ·
                  edit-2
                  5 days ago

                  I can’t speak for the maintainer. What I can say is that FOSS maintainers tend to not like putting up with noise.

                  This is the extent of what they said:

                  This project is not an appropriate arena to advertise your personal politics.

                  Then the github issue got brigaded. Why are there hundreds of reactions to a one-sentence response to a one-word change that doesn’t matter at all? The fact that we’re even having this discussion is crazy imo.

                  It literally does not matter, and I’d prefer their developers write code than engage in a culture war.

                  I don’t know kling’s personal politics, nor do I particularly care. What I care about is the quality of the code in the project, and if they’re able to attract the type of talent that would constructively contribute to the project. A one-off rejection of a noise PR isn’t an indication of issues IMO.