732
Merge then review (programming.dev)
submitted 8 months ago* (last edited 8 months ago) by agilob@programming.dev to c/programmer_humor@programming.dev

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
[-] will_a113@lemmy.ml 219 points 8 months ago

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

[-] RonSijm@programming.dev 77 points 8 months ago

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

[-] Blackmist@feddit.uk 29 points 8 months ago* (last edited 8 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.

[-] Aviandelight@mander.xyz 13 points 8 months ago

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

[-] termus@beehaw.org 18 points 8 months ago

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

load more comments (3 replies)
[-] Quik@infosec.pub 42 points 8 months ago

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

[-] Theharpyeagle@lemmy.world 20 points 8 months ago

Node: "Am I a joke to you?"

[-] Jesus_666@feddit.de 23 points 8 months ago
[-] SpaceNoodle@lemmy.world 21 points 8 months ago
[-] simple@lemm.ee 119 points 8 months ago

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.

[-] NocturnalMorning@lemmy.world 36 points 8 months ago

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

[-] SpeakinTelnet@programming.dev 34 points 8 months ago

Keep everyone awake and on their toes.

[-] EatYouWell@lemmy.world 27 points 8 months ago

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)
[-] pec@sh.itjust.works 89 points 8 months ago

Stop transfering people from sales to engineering!

[-] HowManyNimons@lemmy.world 19 points 8 months ago

But Elon's annoying!

[-] sirdorius@programming.dev 74 points 8 months ago

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)
[-] rainynight65@feddit.de 64 points 8 months ago

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

load more comments (3 replies)
[-] EmergMemeHologram@startrek.website 51 points 8 months ago

No integration is as continuous as editing in prod.

load more comments (3 replies)
[-] notannpc@lemmy.world 47 points 8 months ago

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.

[-] zalgotext@sh.itjust.works 16 points 8 months ago

Another benefit is you never have to worry about merge conflicts

[-] VantaBrandon@lemmy.world 45 points 8 months ago

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

load more comments (3 replies)
[-] SkyNTP@lemmy.ca 43 points 8 months ago* (last edited 8 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.

[-] bleistift2@feddit.de 36 points 8 months ago

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)
[-] Blackmist@feddit.uk 36 points 8 months ago

Do we have a Linked In Lunatics sub on Lemmy?

load more comments (2 replies)
[-] dannym@lemmy.escapebigtech.info 33 points 8 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.

[-] Manifish_Destiny@lemmy.world 30 points 8 months ago* (last edited 8 months ago)

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

[-] VantaBrandon@lemmy.world 29 points 8 months ago

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

load more comments (1 replies)
[-] debil@lemmy.world 25 points 8 months ago

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

load more comments (1 replies)
[-] Sanctus@lemmy.world 24 points 8 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.

[-] okamiueru@lemmy.world 22 points 8 months ago

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.

[-] DudeDudenson@lemmings.world 16 points 8 months ago

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

load more comments (1 replies)
[-] PepeLivesMatter@lemmy.today 22 points 8 months ago

Developers: "Move fast and break things."

Things: break

Developers: surprised Pikachu face

load more comments (1 replies)
[-] RageAgainstTheRich@lemmy.world 21 points 8 months ago

What in the shit is "xtreme programming"?

[-] cypherix93@lemmy.world 23 points 8 months ago

it's NewGame+ for when you 100% programming

load more comments (2 replies)
load more comments (6 replies)
[-] Anders429@programming.dev 21 points 8 months ago

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

[-] silencioso@lemmy.world 20 points 8 months ago

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.

[-] redcalcium@lemmy.institute 14 points 8 months ago

Pair programming? Then the code is already reviewed.

load more comments (1 replies)
load more comments (7 replies)
[-] xmunk@sh.itjust.works 20 points 8 months ago

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

[-] TheFerrango@lemmy.basedcount.com 19 points 8 months ago

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

load more comments (2 replies)
[-] LiveLM@lemmy.zip 19 points 8 months ago

LinkedIn "influencers" are insufferable, dear god

[-] nxdefiant@startrek.website 18 points 8 months ago

I help JavaScript engineers become framework A...

ssholes.

[-] fosforus@sopuli.xyz 17 points 8 months ago* (last edited 8 months ago)

That would certainly explain some things in the nodejs culture.

[-] intensely_human@lemm.ee 17 points 8 months ago

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)
[-] Nighed@sffa.community 15 points 8 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!

[-] DefinitelyNotAPhone@hexbear.net 15 points 8 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.

[-] gatelike@feddit.de 15 points 8 months ago

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

[-] NigelFrobisher@aussie.zone 14 points 8 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.

[-] buh@hexbear.net 13 points 8 months ago

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

load more comments
view more: next ›
this post was submitted on 14 Nov 2023
732 points (97.0% liked)

Programmer Humor

18388 readers
544 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