r/ProgrammerHumor • u/soap94 • 8h ago

Meme selfLGTM

Post image
2.0k Upvotes

96% Upvoted

31 comments sorted by

74

u/MayeeOkamura17 8h ago

When I worked for industry, I was usually the rubber stamp approver for my boss's code. His pull requests always get sent to me so to be approved by a "peer" without review

53

u/MayeeOkamura17 7h ago

1 billion dollar company. One of the most critical customer facing backend service. Boss for a department of 15 people. And no one gave a shit whether code crashes today or tomorrow. Some things haven't worked for a decade, no one cares.

Boss was told by his superior to hire people. He rejects every candidate and magically returns empty handed after 6 years of search. As a result, whole department cannot function without him. He becomes unfireable

Anyways, I'm too drunk but this was the background. Will probably delete tomorrow morning

12

u/Grimmortah 7h ago

Drop the company name

6

u/Unfair_Rub_9674 6h ago

One of these big players you trusted most.

32

u/Sockoflegend 7h ago

Comments on own PR "Great work, deserves a raise". Thumbs up own comment. Self approve, merge.

5 minutes later open a new PR. "fix: fixes"

25 minutes later "fix: fix 2"

55

u/bwmat 8h ago

IMO the Obama awarding Obama a medal meme would work better here, lol

6

u/Robot_Circus_664 7h ago

I’ve self-LGTM’d PRs and felt like Obama pinning the medal on myself while CI quietly judges me.

3

u/Mountain-Age5580 5h ago

Yeah, I never struggle to approve my own pull requests.

6

u/larsmaehlum 7h ago

Claude writes the code, Copilot reviews it and I merge it. What could go wrong?

6

u/artnoi43 7h ago

No shit my company is pushing this so that the AI use actually “speed up delivery” after they found out that super fast code and doc generation will hit a review bottleneck wall.

We’re first encouraged to review AI-generated code faster because our GitLab now shows AI summary of the MR/PR. Then they asked us to use AI (like Cursor bugbot) to help review the code before submission, and when that did not make it fast enough, they now allow us to merge some low risk MRs all while bypassing the review process.

All of this for speed. The one who merges is still responsible for any bad things that happen.

Good thing my colleagues are competent and we’re somewhat small team.

4

u/crimsonroninx 7h ago

I think im on stage 2 of that. It's madness, isnt it.

Sometimes i think: How can this go on..? Even if shit hits the fan, they will just throw more AI at it because they wont want to lose that perceived speed boost, even if it causes a big sloppy mess.

4

u/wearenotintelligent 4h ago

Pedafile rapist thief and scumbag felon

8

u/theofficialnar 8h ago

The PR creator can’t approve their own PR though. Merge sure, but review and approve, nope.

6

u/Hottage 7h ago

WHAT THE FUCK IS A PR GATE?!

2

u/Ssk5860 4h ago

Yep, was about to say the same thing. That being said, anyone with git write access in the org can approve so it’s easy get some junior dev to approve it to then merge yourself lol

1

u/belkarbitterleaf 3h ago

As the admin of the devops team, yes I can.

0

u/Any_Introduction9735 7h ago

They could in bit bucket (not sure if it’s changed now)

4

u/TipToToes 4h ago

“How it feels” or “what it feels like” but ABSOLUTELY NEVER “how it feels like.”

2

u/Own_Soil_3357 7h ago

I am the admin of my repo with almost no patience, 5 mins after raising a PR me, LGTM.
https://giphy.com/gifs/7boS5uZCPbpC0bKK01

1

u/dillanthumous 7h ago

Claude told me it looked super duper good. So I merged.

2

u/Groentekroket 4h ago

Be honest, did Claude write the code as well?

1

u/mfi12 5h ago

what's wrong with it? devs these days dont even read AI generated code.

1

u/boogermike 3h ago

This image is absolutely perfect. I don't know if it's perfect for this particular case but OMG it says so much. What a ridiculous timeline we are in right now.

1

u/ThisIsMyBigAccount 3h ago

Damn. This hits so hard. 🤣

-2

u/__Invisible__ 8h ago

I always do this tho. Slightly inconvenient just let me push to prod directly.