this post was submitted on 18 Sep 2024
619 points (98.1% liked)

Programmer Humor

19187 readers
1172 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 32 comments
sorted by: hot top controversial new old
[–] [email protected] 3 points 5 hours ago (2 children)

What is the original source material for the image and the name of the woman within?

[–] [email protected] 1 points 4 hours ago* (last edited 4 hours ago) (1 children)
[–] [email protected] 2 points 3 hours ago

Thanks, comrade

[–] [email protected] 7 points 4 hours ago* (last edited 4 hours ago) (1 children)

This is the artist, if you can excuse a link to X which I found on reddit via google.

...

I feel dirty.

Edit: Here's a KYM article. No idea why I didn't find it on my first sweep. It might have something to do with google being absolute dogshit these days.

[–] [email protected] 3 points 3 hours ago

Thanks bro

Also I think that's Note wearing a TempleOS shirt in the background of this other post by the artist.

[–] [email protected] 6 points 8 hours ago (1 children)

i don't understand it, am i missing some context here?

[–] [email protected] 15 points 7 hours ago

The meme format implies she catfished you with the promise of "Netflix and chill" at her house only to pull a gun on you.

In particular, she wants you to review and merge that goddamn pull request she made to your open source project repo two months ago that finally fixes that one really annoying bug.

[–] [email protected] 7 points 9 hours ago (1 children)

https://github.com/ZILtoid1991

Please, do pull requests on my projects! I might even accept them.

[–] [email protected] 4 points 6 hours ago (2 children)

You realy want the D don't you

[–] [email protected] 1 points 54 minutes ago
[–] [email protected] 5 points 6 hours ago

gigachad_yes.gif

[–] [email protected] 4 points 13 hours ago

Takes on a slightly different meaning in the context of XZ utils

[–] [email protected] 20 points 21 hours ago (1 children)
[–] [email protected] 18 points 15 hours ago

lunatic girl threatening me

[–] [email protected] 4 points 22 hours ago

First problem is she's expecting a right boob like me to review her pull request. Second problem is I don't give a shit.

[–] [email protected] 4 points 23 hours ago

Babe, you turn me (O)n

[–] [email protected] 18 points 23 hours ago (2 children)

"Did you run the formatter on this?"

Bonus points if it's python code and nowhere in the docs does it say which of the many formatters to use.

[–] [email protected] 3 points 16 hours ago

Just make another PR where you add formatter instructions to the readme and pyproject.toml.

[–] [email protected] 2 points 16 hours ago* (last edited 15 hours ago) (1 children)
[–] [email protected] 4 points 16 hours ago (1 children)

I'm fond of ruff lately. Pretty much the same as black, but it just comes with the linter instead of being separate tools.

[–] [email protected] 2 points 15 hours ago* (last edited 6 hours ago)

👀 I’d fine-tuned (copied and pasted) a pylint setup that had some of black, what are the differences?

Edit: it’s so much faster than black, I’m switching

[–] [email protected] 14 points 23 hours ago (1 children)

The number of comments is inversely proportional to the size of the pull request.

[–] [email protected] 4 points 23 hours ago

Yet the review time is exponential with the size.

[–] [email protected] 15 points 23 hours ago (1 children)

Just pull the trigger, we both know how this ends.

[–] [email protected] 3 points 19 hours ago

Production errors.

[–] [email protected] 34 points 1 day ago (1 children)

I have either written or gotten a variant of every single one of these comments 🫠:

Please include the JIRA task in the commit title.

Did you run any manual testing?

Where's the PRD link in the commit message?

Can you please split this into multiple smaller commits?

Can you combine these two commits?

Did you email Jon about this because he's working on that project with Sarah and you might be duplicating efforts.

This should be named BarFoo instead of FooBar.

Why aren't you using CorporateInternalLib16 that does 90% of this?

Why aren't you using ThirdPartyPaidLibByExEmployee?

Why aren't you using StandardLib thing you forgot existed?

All our I/O should be async.

All our hot loop code needs to be sync.

This will increase latency of NonCoreBusinessFlow by 0.01%. can you shave some time off so we can push in feature B also?

Please add a feature flag so we can do gradual rollout.

What operational levers does this have?

Lgtm - just address those comments

[–] [email protected] 18 points 20 hours ago (1 children)

I dunno, plenty of those sound pretty reasonable.

[–] [email protected] 1 points 4 hours ago

Mostly, yes.

I'd like to find a better way to phrase "why aren't you . . . " questions. It carries an accusatory tone in text, even if you don't intend that. The answer is almost invariably going to be either "I didn't know it existed" or "because reason X". Neither case justifies the accusatory tone. Maybe if the "I didn't know it existed" answer was something so basic that they really should have known it existed, but probably not even then.

[–] [email protected] 6 points 1 day ago

Look at that tiny numstat, I'll finish the review before the CI is even done

[–] [email protected] 73 points 1 day ago

Remove these blank lines.

I’m not seeing unit tests for this.

Unnecessary comment.

BLAM

Ow! Also, this could’ve been a smaller calibur.

[–] [email protected] 48 points 1 day ago (1 children)

LGTM (lunatic gunner targeting me)

[–] [email protected] 7 points 1 day ago

it's me I'm lunatic gunner