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

Programmer Humor

19821 readers
446 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
732
Merge then review (programming.dev)
submitted 1 year ago* (last edited 1 year 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 1 year 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 1 year 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 1 year ago* (last edited 1 year 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 1 year ago (1 children)

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

[–] [email protected] 18 points 1 year 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 1 year ago (1 children)

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

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

Node: "Am I a joke to you?"

[–] [email protected] 23 points 1 year ago
[–] [email protected] 21 points 1 year ago
[–] [email protected] 119 points 1 year 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 1 year ago (3 children)

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

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

Keep everyone awake and on their toes.

[–] [email protected] 27 points 1 year 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 1 year ago (1 children)

Stop transfering people from sales to engineering!

[–] [email protected] 20 points 1 year ago

But Elon's annoying!

[–] [email protected] 74 points 1 year 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 1 year 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 1 year ago (3 children)

No integration is as continuous as editing in prod.

load more comments (3 replies)
[–] [email protected] 47 points 1 year 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 1 year ago

Another benefit is you never have to worry about merge conflicts

[–] [email protected] 45 points 1 year 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 1 year ago* (last edited 1 year 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 1 year ago (2 children)

Do we have a Linked In Lunatics sub on Lemmy?

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

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

[–] [email protected] 29 points 1 year 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 1 year 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 1 year 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 1 year 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 1 year ago (1 children)

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

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

Developers: "Move fast and break things."

Things: break

Developers: surprised Pikachu face

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

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

[–] [email protected] 21 points 1 year ago (7 children)

What in the shit is "xtreme programming"?

[–] [email protected] 23 points 1 year 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 1 year 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 1 year ago (2 children)

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

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

LinkedIn "influencers" are insufferable, dear god

[–] [email protected] 18 points 1 year ago

I help JavaScript engineers become framework A...

ssholes.

[–] [email protected] 17 points 1 year 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 1 year ago* (last edited 1 year ago)

That would certainly explain some things in the nodejs culture.

[–] [email protected] 15 points 1 year ago

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

[–] [email protected] 15 points 1 year 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 1 year 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 1 year 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 1 year 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 ›