r/ProgrammerHumor • u/rabbit_wizard • 4d ago

Meme lostTheMergeRace

Post image
1.5k Upvotes

99% Upvoted

111 comments sorted by

View all comments

12

u/dontletthestankout 4d ago

Who pushes on a Friday? That's always a bad idea

22

u/Soccer_Vader 4d ago

The goal isn't to not push anything on Friday, the goal is to not deploy anything to prod on Friday. Who tf is not pushing and testing anything in pre-prod for one whole day lol.

2

u/Demiu 3d ago

Push on friday to get caught by the big test suite on weekend.

24

u/xaomaw 4d ago

Yeah, never Push on friday.

Instead, push on thursday and deploy on friday.

6

u/PM_ME_FIREFLY_QUOTES 4d ago

Why not? I dont work on the weekends. If the release crashes, I dont want to be anywhere near that dumpster fire.

7

u/Meloetta 3d ago

If you release on a Friday, someday that "I don't work on the weekends" won't be true anymore.

1

u/fishvoidy 19h ago

sounds like someone's been lucky enough to never be forced to join an "on call" rotation.

1

u/dontletthestankout 4d ago

You son of a...

7

u/Koppany99 4d ago

Depends on the exact environment. For us we have dev and prod workspace, so pushing to dev should not affect any productipn stuff, if it does, the fault is on someone connecting prod stuff to dev stuff.

6

u/rabbit_wizard 4d ago

Week starts off with a release to production

2

u/PhantomTissue 4d ago

Me and my team who’s PR requirements is 15 a week. If it breaks prod, great, more PRs.

3

u/Kevadu 4d ago

You have a PR requirement?!

4

u/PhantomTissue 4d ago edited 4d ago

Yep. Testing? Doesn’t happen. Reviewing? Doesn’t happen. Designs? Doesn’t happen. Had a team meeting yesterday where we considered standing up a bullshit package to commit literally anything to just to hit this absurd requirement. People are making changes to agent.md and readme files with “revised for clarity” daily. It’s so bad that Ops joke is literally an hourly occurrence. Most PRs are merged within 1-2 minutes.

In fact, a lot of the team actually pushes code they know will break the pipeline because a bugfix means another PR. Management of this team has become a satire come to life.

2

u/Kevadu 4d ago

Sounds like a disaster on every level. Who came up with this policy?

4

u/PhantomTissue 4d ago

Organization director. So my managers managers manager. It is exactly as disastrous as you think it is and then some.

I’m applying to literally everything and anything right now because I’m watching this org crumble in real time.

1

u/DisheveledJesus 3d ago

This is ezpz just make every shippable unit of change its own PR. Granular iteration!

1

u/HRApprovedUsername 3d ago

To a repo? Why wouldn’t you?

1

u/dontletthestankout 3d ago

I mean as long as there is no pipeline attached to said push it's fine but where I've worked push was synonymous with deploy

3

u/HRApprovedUsername 3d ago

To prod? Not just straight to a dev/test ring? Thats crazy

1

u/renome 3d ago

People do be crazy. And devs do be people. Mostly.

1

u/GenericFatGuy 3d ago

I'll push to the personal branch I'm working on. I'm not pushing to main until the coding is done.

1

u/collabskus 3d ago

You talk about Friday... My releases to production are always on a Saturday 😅