this post was submitted on 05 Nov 2024
878 points (98.8% liked)

Programmer Humor

19813 readers
191 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 2 years ago
MODERATORS
 
top 50 comments
sorted by: hot top controversial new old
[–] [email protected] 280 points 1 month 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.

[–] [email protected] 54 points 1 month ago (3 children)

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

[–] [email protected] 57 points 1 month 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)
load more comments (2 replies)
[–] [email protected] 43 points 1 month 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] 16 points 1 month ago* (last edited 1 month ago)

Commit with Co-authored-by: Copilot

or maybe better --author=Copilot

It would certainly help evaluate submissions to have that context

[–] [email protected] 156 points 1 month ago (14 children)

We will never solve the Scunthorpe Problem.

[–] [email protected] 47 points 1 month ago (1 children)
load more comments (1 replies)
[–] [email protected] 26 points 1 month ago (5 children)

Hasn't it been proven unsolvable?

[–] [email protected] 54 points 1 month ago (2 children)

Impossible. There is always some mf named like cum-sock, smh

[–] [email protected] 34 points 1 month ago

some mf named like cum-sock

Excuse me? My family BUILT this country!

load more comments (1 replies)
load more comments (4 replies)
load more comments (12 replies)
[–] [email protected] 131 points 1 month 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] 134 points 1 month ago (1 children)

Gotta love Microsoft Power Bisexual

[–] [email protected] 37 points 1 month ago (3 children)

I now identify as a Power Bisexual.

load more comments (3 replies)
[–] [email protected] 47 points 1 month ago (2 children)

How is that inconsiderate? That's just informal

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

[–] [email protected] 63 points 1 month 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 1 month ago

I'm confused how bi is inappropriate

load more comments (2 replies)
[–] [email protected] 86 points 1 month 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

[–] [email protected] 70 points 1 month ago (1 children)

Those commit messages though 🤣

[–] [email protected] 50 points 1 month ago (1 children)

They automated randomization of the commit messages? Wtf?

[–] [email protected] 37 points 1 month ago

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

[–] [email protected] 65 points 1 month ago* (last edited 1 month ago)

oh god

yeah, no. haha

[–] [email protected] 34 points 1 month ago
[–] [email protected] 32 points 1 month ago* (last edited 1 month 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..

[–] Grass 16 points 1 month ago

wtf it was real?

[–] [email protected] 56 points 1 month ago (1 children)
load more comments (1 replies)
[–] [email protected] 50 points 1 month ago

It’s a clbuttic mistake.

[–] [email protected] 49 points 1 month ago (1 children)
[–] [email protected] 25 points 1 month ago

Google: kill child process
FBI: ಠ_ಠ

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

[–] [email protected] 42 points 1 month 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.

[–] [email protected] 28 points 1 month 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 😢

[–] [email protected] 14 points 1 month ago

But think of the gains!

load more comments (1 replies)
load more comments (2 replies)
[–] [email protected] 33 points 1 month ago (1 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 1 month ago* (last edited 1 month 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.

[–] [email protected] 30 points 1 month ago (1 children)

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

[–] [email protected] 12 points 1 month ago

Or just have some random subset of browsers support them for some reason and other browsers not so much. It’s the html way.

[–] [email protected] 30 points 1 month ago

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

[–] [email protected] 29 points 1 month ago

Fucking Scunthorpes!

[–] [email protected] 29 points 1 month ago (2 children)

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

[–] [email protected] 80 points 1 month 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)
[–] omgitsaheadcrab 29 points 1 month ago

It's got an arse in there

[–] [email protected] 26 points 1 month ago (1 children)

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

load more comments (1 replies)
[–] [email protected] 22 points 1 month ago* (last edited 1 month ago) (1 children)

Ok, how is "charset" vulgar?

Edit: got it; arse

load more comments (1 replies)
[–] [email protected] 20 points 1 month ago

Luckily the feck attribute is too obscure to be in the line of fire.

[–] [email protected] 20 points 1 month ago (2 children)

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

load more comments (2 replies)
load more comments
view more: next ›