898
top 50 comments
sorted by: hot top controversial new old
[-] Ephera@lemmy.ml 285 points 1 month ago

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.

[-] TrickDacy@lemmy.world 53 points 1 month ago

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

[-] unexposedhazard@discuss.tchncs.de 57 points 1 month ago

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)
[-] GreenKnight23@lemmy.world 44 points 1 month ago

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)
[-] Kissaki@programming.dev 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

[-] ilinamorato@lemmy.world 158 points 1 month ago

We will never solve the Scunthorpe Problem.

[-] SatouKazuma@programming.dev 26 points 1 month ago

Hasn't it been proven unsolvable?

[-] Valmond@lemmy.world 54 points 1 month ago

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

[-] ImplyingImplications@lemmy.ca 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)
[-] oce@jlai.lu 131 points 1 month ago

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.

[-] gnutrino@programming.dev 135 points 1 month ago

Gotta love Microsoft Power Bisexual

[-] watersnipje@lemmy.blahaj.zone 38 points 1 month ago

I now identify as a Power Bisexual.

load more comments (3 replies)
[-] communism@lemmy.ml 47 points 1 month ago

How is that inconsiderate? That's just informal

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

[-] marcos@lemmy.world 63 points 1 month ago

No, it's right.

Business intelligence is inconsiderate and must be stopped!

[-] Carighan@lemmy.world 12 points 1 month ago

Fully agreed! And while we're at it, get rid of Power Pivot!

load more comments (1 replies)
[-] kittenzrulz123@lemmy.blahaj.zone 29 points 1 month ago

I'm confused how bi is inappropriate

load more comments (2 replies)
[-] Zikeji@programming.dev 86 points 1 month ago

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

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

[-] dohpaz42@lemmy.world 70 points 1 month ago

Those commit messages though 🤣

[-] jaybone@lemmy.world 50 points 1 month ago

They automated randomization of the commit messages? Wtf?

[-] ByteJunk@lemmy.world 37 points 1 month ago

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

[-] FiskFisk33@startrek.website 65 points 1 month ago* (last edited 1 month ago)

oh god

yeah, no. haha

[-] lily33@lemm.ee 34 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..

[-] maniel@sopuli.xyz 34 points 1 month ago
[-] Grass@sh.itjust.works 16 points 1 month ago

wtf it was real?

[-] schnurrito@discuss.tchncs.de 57 points 1 month ago
load more comments (1 replies)
[-] dsilverz@thelemmy.club 52 points 1 month ago
[-] hakunawazo@lemmy.world 25 points 1 month ago

Google: kill child process
FBI: ಠ_ಠ

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

[-] deegeese@sopuli.xyz 51 points 1 month ago

It’s a clbuttic mistake.

[-] brygphilomena@lemmy.world 42 points 1 month ago

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@lemmy.world 28 points 1 month ago

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 😢

[-] Lemminary@lemmy.world 14 points 1 month ago

But think of the gains!

load more comments (1 replies)
load more comments (2 replies)
[-] NumerousGeorg@sopuli.xyz 33 points 1 month ago

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....

[-] dan@upvote.au 43 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.

[-] Gork@lemm.ee 30 points 1 month ago

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

load more comments (1 replies)
[-] antonim@lemmy.dbzer0.com 30 points 1 month ago

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

[-] Passerby6497@lemmy.world 29 points 1 month ago

Fucking Scunthorpes!

[-] Valmond@lemmy.world 29 points 1 month ago

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

[-] Scoopta@programming.dev 81 points 1 month ago

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@sh.itjust.works 29 points 1 month ago

It's got an arse in there

[-] rainerloeten@lemmy.world 26 points 1 month ago

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

load more comments (1 replies)
[-] MonkderVierte@lemmy.ml 23 points 1 month ago* (last edited 1 month ago)

Ok, how is "charset" vulgar?

Edit: got it; arse

load more comments (1 replies)
[-] Taleya@aussie.zone 20 points 1 month ago

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

load more comments (2 replies)
[-] Flamekebab@piefed.social 20 points 1 month ago

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

load more comments
view more: next ›
this post was submitted on 05 Nov 2024
898 points (98.8% liked)

Programmer Humor

19774 readers
31 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