this post was submitted on 14 Nov 2023
732 points (97.0% liked)

Programmer Humor

19480 readers
175 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
732
Merge then review (programming.dev)
submitted 11 months ago* (last edited 11 months ago) by [email protected] to c/[email protected]
 

Move fast and break things.
Merge vulnerabilities.
Double the work.
Merge code without tests.
Anything, but don't let code become stale.

top 50 comments
sorted by: hot top controversial new old
[–] [email protected] 219 points 11 months ago (2 children)

Having a hard time determining whether this is sarcasm or not. Then I see the phrase "JavaScript Engineer" and become doubly confused.

[–] [email protected] 77 points 11 months ago (5 children)

I don't think it's satire, this guy is actively defending this on Linkedin: https://i.imgur.com/SlJPG85.png

[–] [email protected] 29 points 11 months ago* (last edited 11 months ago)

I distinguish four types. There are clever, hardworking, stupid, and lazy officers. Usually two characteristics are combined. Some are clever and hardworking; their place is the General Staff. The next ones are stupid and lazy; they make up 90 percent of every army and are suited to routine duties. Anyone who is both clever and lazy is qualified for the highest leadership duties, because he possesses the mental clarity and strength of nerve necessary for difficult decisions. One must beware of anyone who is both stupid and hardworking; he must not be entrusted with any responsibility because he will always only cause damage.

-- Kurt von Hammerstein

LinkedIn is Facebook for that last type.

[–] [email protected] 13 points 11 months ago (1 children)

That's a relief because I thought I'd stumbled into LinkedIn Lunatics for a hot second.

[–] [email protected] 18 points 11 months ago

Linkedin is for lunatics. Just a bunch of goobers giving digital handjobs to each other.

load more comments (3 replies)
[–] [email protected] 42 points 11 months ago (1 children)

I think the latter makes clear that this is a joke account, doesn’t it?

[–] [email protected] 20 points 11 months ago (2 children)

Node: "Am I a joke to you?"

[–] [email protected] 23 points 11 months ago
[–] [email protected] 21 points 11 months ago
[–] [email protected] 119 points 11 months ago (3 children)

Having to go through the process of merging hurts morale and slows performance. Give everyone on your team the right to force push to master.

[–] [email protected] 36 points 11 months ago (3 children)

Yes, especially the newbies who don't know what they're doing.

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

Keep everyone awake and on their toes.

[–] [email protected] 27 points 11 months ago (3 children)

You're not truly part of the team until you cause a massive outage.

load more comments (3 replies)
load more comments (2 replies)
load more comments (2 replies)
[–] [email protected] 88 points 11 months ago (1 children)

Stop transfering people from sales to engineering!

[–] [email protected] 20 points 11 months ago

But Elon's annoying!

[–] [email protected] 74 points 11 months ago (2 children)

I really wish LinkedIn would add an anonymous cringe emoji. I would use it on like 90% of the content on that site.

load more comments (2 replies)
[–] [email protected] 64 points 11 months ago (3 children)

I'm having a hard time figuring out whether this guy is a fucking moron or a fucking idiot.

load more comments (3 replies)
[–] [email protected] 51 points 11 months ago (3 children)

No integration is as continuous as editing in prod.

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

Amateur. You want real performance? Code in prod. Literally could not be better for collaboration to have the whole team working directly from production servers. Best part? You get INSTANT feedback.

[–] [email protected] 16 points 11 months ago

Another benefit is you never have to worry about merge conflicts

[–] [email protected] 45 points 11 months ago (3 children)

I just commit directly to master with auto-deploy like a real cowboy, yee-haw!

load more comments (3 replies)
[–] [email protected] 43 points 11 months ago* (last edited 11 months ago)

What does "stale code" even mean in this context?

Does that mean it falls behind stable? Just merge stable into your branch; problem solved.

Or is this just some coded language for "people aren't adopting my ideas fast enough". Stop bitching and get good.

[–] [email protected] 36 points 11 months ago (2 children)

Do we have a Linked In Lunatics sub on Lemmy?

load more comments (2 replies)
[–] [email protected] 36 points 11 months ago (1 children)

My old boss (at a sturtup with some ten ppl) loved to do this. When you’re done with your work, merge to master. Boss-man would then revert the commits if he didn’t like the result. Since the branches all were merged, no-one knew what was actually in prod. Fun times.

load more comments (1 replies)
[–] [email protected] 33 points 11 months ago

If somebody actually did that it would be grounds for removing their privileges to merge into master. THIS, THIS is why the JavaScript ecosystem has gotten so bad, people with mentalities similar to his.

[–] [email protected] 30 points 11 months ago* (last edited 11 months ago)

'i help JavaScript engineers become framework architects by getting them forcibly reassigned.'

[–] [email protected] 29 points 11 months ago (1 children)

Better yet just edit files live on prod from Notepad (not plus plus) over Samba for "xtreme moral" boost

load more comments (1 replies)
[–] [email protected] 25 points 11 months ago (1 children)

I dunno but xtreme programming sounds like something straight outta Musk's wettest teenage day dreams.

load more comments (1 replies)
[–] [email protected] 24 points 11 months ago

This is why I include those preservative libraries in my projects. My code doesn't go stale for a whole three weeks longer.

[–] [email protected] 22 points 11 months ago (1 children)

This is satire, right? Surely no one would put their name on that publicly?

Like someone working in a kitchen boasting about a life hack of not wasting time with hygiene.

[–] [email protected] 16 points 11 months ago (1 children)

Wash your hands after cooking, never let food products sit stale

load more comments (1 replies)
[–] [email protected] 22 points 11 months ago (1 children)

Developers: "Move fast and break things."

Things: break

Developers: surprised Pikachu face

load more comments (1 replies)
[–] [email protected] 21 points 11 months ago

Bet you $50 we later learn this guy was orchestrating a supply chain attack.

[–] [email protected] 21 points 11 months ago (7 children)

What in the shit is "xtreme programming"?

[–] [email protected] 23 points 11 months ago (2 children)

it's NewGame+ for when you 100% programming

load more comments (2 replies)
load more comments (6 replies)
[–] [email protected] 20 points 11 months ago (1 children)

At my company we're so agile that we directly deploy branches from developers' local machines to customers for A/B testing.

[–] [email protected] 19 points 11 months ago (2 children)

Call it “container orchestration” and charge an extra 20% to the customer

load more comments (2 replies)
[–] [email protected] 20 points 11 months ago (8 children)

Before everyone loses their minds, in Extreme Programming there are safeguards other than PR reviews. Before you submit a PR, you are supposed to have written the tests and to have written your code with pair programming, so your code already has some safety measures in place. On top of that, when you merge and deploy, more tests are run, and only if all of them are green do your changes go into production.

[–] [email protected] 14 points 11 months ago (1 children)

Pair programming? Then the code is already reviewed.

load more comments (1 replies)
load more comments (7 replies)
[–] [email protected] 19 points 11 months ago

LinkedIn "influencers" are insufferable, dear god

[–] [email protected] 18 points 11 months ago

I help JavaScript engineers become framework A...

ssholes.

[–] [email protected] 17 points 11 months ago (1 children)

If you’re working in a context where it’s okay to make mistakes so long as they get fixed later, you’re not working on anything important.

load more comments (1 replies)
[–] [email protected] 17 points 11 months ago* (last edited 11 months ago)

That would certainly explain some things in the nodejs culture.

[–] [email protected] 15 points 11 months ago

this made my heart rate go up a little bit in a way that doesn't feel good

[–] [email protected] 15 points 11 months ago

Nothing improves morale like the on-call having to unfuck production for the third time that hour because mUh VeLoCiTy decided code review and testing in CI was too slow.

Techbros are fucking cultists.

[–] [email protected] 15 points 11 months ago

Kinda acceptable if you have a slow release cadence. Everything needs to be reviewed and fixed/accepted (with defect/US raised) before production though.

Needs to be in a smaller team with decent Devs too though!

[–] [email protected] 13 points 11 months ago

What if instead of continuous integration we had continuous Disintegration, where you code while listening to The Cure on repeat

[–] [email protected] 13 points 11 months ago

It’s insane to me that gitflow won over TBD and Continuous Integration to the point that this is now considered an extreme position. Not all projects are open source with many remote collaborators.

load more comments
view more: next ›