this post was submitted on 05 Nov 2024
869 points (99.0% liked)

Programmer Humor

19488 readers
1091 users here now

Welcome to Programmer Humor!

This is a place where you can post jokes, memes, humor, etc. related to programming!

For sharing awful code theres also Programming Horror.

Rules

founded 1 year ago
MODERATORS
 
top 50 comments
sorted by: hot top controversial new old
[–] [email protected] 265 points 2 days ago (3 children)

Tangentially related rant: We had a new contributor open up a pull request today and I gave their changes an initial look to make sure no malicious code is included.
I couldn't see anything wrong with it. The PR was certainly a bit short, but the task they tackled was pretty much a matter of either it works or it doesn't. And I figured, if they open a PR, they'll have a working solution.

...well, I tell the CI/CD runner to get going and it immediately runs into a compile error. Not an exotic compile error, the person who submitted the PR had never even tried to compile it.

Then it dawned on me. They had included a link to a GitHub Copilot workspace, supposedly just for context.
In reality, they had asked the dumbass LLM to do the change described in the ticket and figured, it would produce a working PR right off the bat. No need to even check it, just let the maintainer do the validation.

In an attempt to give them constructive feedback, I tried to figure out, if this GitHub Copilot workspace thingamabob had a Compile-button that they just forgot to click, so I actually watched Microsoft's ad video for it.
And sure enough, I saw right then and there, who really was at fault for this abomination of a PR.

The ad showed exactly that. Just chat a bit with the LLM and then directly create a PR. Which, yes, there is a theoretical chance of this possibly making sense, like when rewording the documentation. But for any actual code changes? Fuck no.

So, most sincerely: Fuck you, Microsoft.

[–] GreenKnight23 38 points 2 days ago (3 children)

dude. i feel that pain.

i got a dev fired because they absolutely refused to test their changes before submitting.

I'm not talking once or twice either. at least a year of that bullshit. i had to show my boss how many hours of wasted time it was taking me because I look at the code first, like literally anybody. Eventually boss pipd them and fired them but holy fuck i wanted to kick that douche in the groin every time i saw a pr with their name on it.

next place I work I'm insisting on a build step success to assign a pr.

load more comments (3 replies)
[–] [email protected] 15 points 2 days ago* (last edited 2 days ago)

Commit with Co-authored-by: Copilot

or maybe better --author=Copilot

It would certainly help evaluate submissions to have that context

[–] TrickDacy 52 points 2 days ago (3 children)

Surely you have to blame the idiot human here who actually has the ability to reason (in theory)

[–] [email protected] 55 points 2 days ago (5 children)

You think the decision to build this bot like that was not made by a human? Its idiot humans all the way down.

load more comments (5 replies)
[–] [email protected] 10 points 2 days ago

Well, for reasons, I happen to know that this person is a student, who has effectively no experience dealing with real-world codebases.

It's possible that the LLM produced good results for the small codebases and well-known exercises that they had to deal with so far.

I'm also guessing, they're learning what a PR is for the first time just now. And then being taught by Microsoft that you can just fire off PRs without a care in the world, like, yeah, how should they know any better?

load more comments (1 replies)
[–] rainerloeten 23 points 2 days ago (1 children)

OMG this took me way too long to get. They replace the substring "ass" 😭😭

[–] pyre 8 points 2 days ago

and "arse", as seen in charset

[–] ilinamorato 154 points 3 days ago (21 children)

We will never solve the Scunthorpe Problem.

[–] GeorgimusPrime 46 points 3 days ago (1 children)
load more comments (1 replies)
[–] [email protected] 8 points 2 days ago

there's a very trivial solution that always works actually, it's called "stop being a prude"

load more comments (19 replies)
[–] [email protected] 126 points 3 days ago (5 children)

I had a Pycharm linter with "inconsiderate writing list" flag my use of "bi" as inappropriate, recommending to use "bisexual" instead. In my data job, BI, means business intelligence, it's everywhere.

[–] [email protected] 131 points 3 days ago (1 children)

Gotta love Microsoft Power Bisexual

[–] [email protected] 36 points 2 days ago (3 children)

I now identify as a Power Bisexual.

load more comments (3 replies)
[–] [email protected] 46 points 3 days ago (2 children)

How is that inconsiderate? That's just informal

(Using "bi" to mean "bisexual", I mean, not "business intelligence" lol)

[–] marcos 63 points 3 days ago (1 children)

No, it's right.

Business intelligence is inconsiderate and must be stopped!

load more comments (1 replies)
load more comments (1 replies)
[–] [email protected] 29 points 3 days ago

I'm confused how bi is inappropriate

load more comments (2 replies)
[–] [email protected] 84 points 3 days ago (5 children)

Holy shit, 10,000 commits because each change was individual (I'm assuming automated).

https://github.com/mrdoob/three.js/pull/29798

[–] dohpaz42 67 points 3 days ago (1 children)

Those commit messages though 🤣

[–] jaybone 50 points 3 days ago (1 children)

They automated randomization of the commit messages? Wtf?

[–] ByteJunk 37 points 2 days ago

Gotta appreciate the level of commitment on this commit...

[–] [email protected] 63 points 3 days ago* (last edited 3 days ago)

oh god

yeah, no. haha

[–] [email protected] 34 points 3 days ago
[–] [email protected] 31 points 3 days ago* (last edited 3 days ago)

And they're all with different commit message:

"switched arse to bottom to create a more uplifting vibe"

"took arse out and put bottom in to keep my language warm and friendly"

"thought bottom would sound a lot nicer than arse, so I used it"

And so on..

load more comments (1 replies)
[–] VantaBrandon 10 points 2 days ago

Thank you so much I so desperately needed that chuckle right now

[–] [email protected] 32 points 2 days ago (4 children)

I am in doubt. That wouldn't even compile. But who am I to think somebody changing something like this would actually do a test compilation afterwards....

[–] [email protected] 42 points 2 days ago* (last edited 2 days ago)

HTML isn't compiled, and unknown attributes are allowed. The best practice is to prefix non-standard attributes with data- (e.g. <div data-foo="test">) but nothing enforces that. Custom attributes can be retrieved in JavaScript or targeted in CSS rules.

load more comments (3 replies)
[–] [email protected] 29 points 2 days ago

clbottomt when the chtopt shows up [imagine this as that popular GIF meme]

[–] [email protected] 9 points 2 days ago (2 children)

I have a friend that works at Salesforce, he told me that they made him change the name of some classes that used the term Blacklist because of inclusivity

[–] steeznson 5 points 2 days ago (1 children)

This kind of thing is infuriating. Especially when the actual etymology of "blacklist" is a 17th century play where a list of nobles scheming against the British monarchy is described as a blacklist.

[–] [email protected] 5 points 2 days ago

But the word black is used so clearly it's racist, right? How can we be woke if we don't randomly ban words because of their alternate meanings?

[–] AusatKeyboardPremi 5 points 2 days ago* (last edited 2 days ago)

My previous workplace did the same thing around 2020 with the words whitelist and blacklist and some other words.

It was around the same time when there was news about GitHub moving from master to main/mainline as the default Git branch.

[–] [email protected] 48 points 3 days ago (1 children)
[–] hakunawazo 25 points 2 days ago

Google: kill child process
FBI: ಠ_ಠ

Google: kill child process linux console
FBI:(︶︿︶)

[–] [email protected] 56 points 3 days ago (1 children)
load more comments (1 replies)
[–] brygphilomena 42 points 3 days ago (3 children)

I've been tempted to create a bot that does nothing but search comments in code for misspelled words and create pull requests for them.

If it stays in comments, little chance in breaking a working codebase and I'd have an insane amount of commits and contributions to a wide variety of codebases for my resume.

I'll never be a top tier coder. But I might make management.

[–] Keenuts 28 points 2 days ago (2 children)

In case that wasn’t satire, please don’t 🥲 A small typo in a comment is not a big issue, and even if the PR is straightforward, a maintainer still has to take some time reviewing it, which takes time away from fixing actual bugs 😢

load more comments (2 replies)
load more comments (2 replies)
[–] [email protected] 50 points 3 days ago

It’s a clbuttic mistake.

[–] Passerby6497 29 points 2 days ago

Fucking Scunthorpes!

[–] [email protected] 21 points 2 days ago (2 children)

The ol' master/ slave configuration again....

[–] [email protected] 8 points 2 days ago

those are terms, this is substrings within words

I haven't seen branches or variables being called arse

Then again, I do like to catch exceptions as up so I can throw up

load more comments (1 replies)
[–] [email protected] 30 points 3 days ago (1 children)

It's time for chbottomt and clbottom to finally become valid HTML statements.

load more comments (1 replies)
[–] Valmond 29 points 3 days ago (2 children)

What is a 'charset' in this depraved persons mind? A corset? Must be a mighty kinky corset.

[–] [email protected] 80 points 3 days ago (3 children)

It's replacing all instances of arse and ass with bottom...but doing so in about the most naïve way possible.

load more comments (3 replies)
[–] [email protected] 29 points 3 days ago

It's got an arse in there

load more comments
view more: next ›