this post was submitted on 28 Feb 2025
581 points (95.2% liked)

Technology

63547 readers
2641 users here now

This is a most excellent place for technology news and articles.

Our Rules

  1. Follow the lemmy.world rules.
  2. Only tech related content.
  3. Be excellent to each other!
  4. Mod approved content bots can post up to 10 articles per day.
  5. Threads asking for personal tech support may be deleted.
  6. Politics threads may be removed.
  7. No memes allowed as posts, OK to post as comments.
  8. Only approved bots from the list below, to ask if your bot can be added please contact us.
  9. Check for duplicates before posting, duplicates may be removed
  10. Accounts 7 days and younger will have their posts automatically removed.

Approved Bots

founded 2 years ago
MODERATORS
L3s@lemmy.world
enu@lemmy.world
technopagan@lemmy.world
L4s@lemmy.world
581
GitHub - LadybirdBrowser/ladybird: Truly independent web browser (github.com)
submitted 3 days ago by mesamunefire@lemmy.world to c/technology@lemmy.world
190 comments fedilink hide all child comments
you are viewing a single comment's thread
view the rest of the comments
[–] RayJW@sh.itjust.works 30 points 3 days ago (3 children)

https://mkultra.monster/tech/2024/07/03/serenityos-and-ladybird

This was a little „write-up“ back when everything became more public.

[–] sugar_in_your_tea@sh.itjust.works 28 points 2 days ago* (last edited 2 days ago) (2 children)

I'm surprised this got any kind of attention.

Here's the turn of events from my perspective:

  1. Someone submits a 1-line PR changing the gender used in a code comment
  2. PR rejected on the grounds that the change is "politically motivated"
  3. Submitter got mad, and proposed removing the rule against "politically motivated" changes, calling it "white supremacist," which is closed
  4. Someone wrote a blog post about it

Here's my analysis:

  1. Stupid change - don't make PRs that simply correct an irrelevant typo in a comment somewhere; some people do this to put stuff on a resume (look at how much FOSS work I do!), and it just wastes everyone's time
  2. Stupid response - it should've been rejected because it's a useless change, not because it's "politically motivated"
  3. Stupid proposal - do you really want to waste a bunch of time fighting over wording in a comment? Because that's the kind of crap you get without a rule like this.
  4. This is all about an irrelevant change to a comment? Why is this getting so much attention?
[–] pr0sp3kt@lemmy.dbzer0.com 7 points 2 days ago (1 children)

I should be an idiot. I dont see a direct relationship between race and sexual orientation. Even if the PR was rejected because a pronounce how the hell this is white supremacist?

[–] sugar_in_your_tea@sh.itjust.works 6 points 2 days ago

Well, didn't the Nazis also discriminate against gay people?

That said, it's a massive leap to go from "rejects 1 line PR that only changes gender in a comment" to literal Nazi...

[–] surewhynotlem@lemmy.world 2 points 2 days ago (1 children)

"comments must be accurate," is not a rule you should bend. Bending it even a little leads to last programming and shit code.

[–] sugar_in_your_tea@sh.itjust.works 7 points 2 days ago* (last edited 2 days ago) (1 children)

True, but that only applies if it's misleading. For example:

// pythagoran theorem 
distance = abs(p2.x - p1.x) + abs(p2.y - p1.y);

Fixing that makes sense because it's wrong and misleading (it's actually Manhattan distance), and a quick glace is insufficient to tell the difference.

But fixing a typo or something that wouldn't be confusing is just noise and should only be fixed with other changes. For example, I intentionally misspelled Pythagorean in my comment above, fixing that to be the right spelling would be a useless change, even if the distance formula used the hypotenuse. It wouldn't be an unreasonable policy to reject PRs that only fix spelling or similar to reduce noise for the maintainers.

[–] surewhynotlem@lemmy.world 4 points 2 days ago (1 children)

Yep, I understand but disagree. Maybe it comes from working with so many ESL coders, but I'll happily accept typo corrections because it's not always obvious what words should be if you're not steeped in the culture.

[–] sugar_in_your_tea@sh.itjust.works 4 points 2 days ago* (last edited 2 days ago) (1 children)

It really depends on the project.

If you're a larger project, you can see a ton of these from people hoping to land a commit to put "contributor to X" on a resume somewhere. Those add up and are really distracting and possibly automated. They waste everyone's time, especially if they spawn a bunch of conversion like this did.

If you're a smaller project, it doesn't matter as much. I work with ESL coders too, so I get it (1/4 of my office is ESL immigrants, and ~2/3 of the broader team is ESL). I fix comments all the time, I just include them with other changes.

So it depends. But in general, a high profile project should reject this noise to discourage this behavior.

[–] pogmommy@lemmy.blahaj.zone 4 points 2 days ago (1 children)

In theory that's fair reasoning. Unfortunately the dev made it clear that his reasoning was based on politics

[–] sugar_in_your_tea@sh.itjust.works 1 points 2 days ago

Did he? I only saw him point to the rule against politics.

He should have said it's because the PR isn't worth the time, but it also seems motivated by something that's against the rules (i.e. why make a PR that only fixes gender in one comment? There was a later PR that was accepted that fixed it in several places).

So without more evidence, I cannot say what the dev's motivations for rejecting the PR were, aside from the apparent rule breakage mentioned. They didn't say they disagreed with the change (i.e. that the change was wrong), just the proposal of the change (i.e. seems more motivated by virtue signaling instead of improving the dev experience). And you can look at the comments and see justification for that position, since it quickly devolved into actual politics with people accusing the dev of being a Nazi.

Maybe if you showed a pattern across more than just this incident (i.e. over months or years), but this sounds more like people being stubborn than tolerant.

[–] DreamlandLividity@lemmy.world 25 points 3 days ago* (last edited 3 days ago)

"We don't accept ideologically motivated changes" = White supremacist language... Yeah, sounds about like what I expected...

[–] gon@lemm.ee 10 points 3 days ago

Thank you for sharing.