r/ProgrammerHumor • u/East-Experience2862 • 15h ago

Meme nothingRunsUntilNoon

Post image
205 Upvotes

87% Upvoted

33 comments sorted by

64

u/faultydesign 15h ago

Who the hell commits code with syntax errors

Ludicrous

43

u/smartgenius1 15h ago

Also: who the hell builds a CI/CD pipeline that allows syntax errors to even be committed in the first place

18

u/Particular-Yak-1984 15h ago

Also, modern IDEs highlight and can basically fix syntax errors for you. So 7 syntax errors would take about 5 mins 

1

u/Frost-Freak 15h ago

Uh my ex coworkers didn't give a damn about IDE highlightings

4

u/burakdobur 12h ago

Ahh, this explains the ex-.

4

u/Frost-Freak 12h ago

I left the company. They're still there. But yeah, that was one of many reasons to leave :)

0

u/meneldal2 9h ago

Only when your IDEs can deal with whatever mess of a build system you have.

I can tell you they are not big fans of ad-hoc make scripts

And while I would love to change the build system I doubt it is happening at this point.

1

u/blooping_blooper 5h ago

right? It shouldn't even make it to review, let alone get merged

18

u/kriseev 12h ago

Are you in an infinitely deep nested dream? Because you keep waking up but never go back to sleep

2

u/deceze 11h ago

Codeception!

36

u/Micromize 15h ago

Meme from the past. 

2

u/ineyy 10h ago

It implies also that he fixes them while sleeping

1

u/_Ganon 10m ago

And then wakes up to higher level of inception ... or does he?

6

u/schewb 9h ago

What the fuck workflow allows your coworker to change your code while you're asleep with you expected to finish their work?

1

u/Fit-Notice-1248 8h ago

Having 98% of the dev team offshore/contractors

1

u/schewb 8h ago

And they just....commit in-progress shit to the same branch as a dev in a different time zone?

1

u/Fit-Notice-1248 8h ago

They create and merge their own PRs that are faulty. Mix this in with non technical managers who think contractors with 50 page resumes are legit and.. yeah pretty much powerless

2

u/schewb 8h ago

They merge with syntax errors? Stupid bugs I understand, but I have a really difficult time believing this happens and actual companies continue to use the contractors.

1

u/Fit-Notice-1248 8h ago

Well I guess in context of this post it isn't necessarily syntax errors but definitely build errors or code that is faulty in general (wrong implementation, doesn't even run locally). These are all pains I've had to deal with and currently dealing with. I put gating up to block these issues but they don't even check the reports to see if shit is broken they just go 🤷🏽

2

u/Sw429 3h ago

Sounds like you need to set up continuous integration tests and branch protections. Don't let anything merge until it meets the standards. You're not powerless here, this is a solved problem.

1

u/Fit-Notice-1248 3h ago

Definitely not powerless but overwhelmed. It's about 12 offshore engineers and I'm just limited in the amount of time I can do these things without losing my sanity.

4

u/johnschnee 13h ago

Why the fuck does your coworker commit syntax errors? Why do you need to fix them?

Build pipeline crashes on PR (if even possible) -> reject or wait for author…

Consider quitting your company if you need to do this

3

u/rosuav 12h ago

What I suspect you may be missing here is that, after fixing a bunch of easy errors, this person wakes up. It's a nightmare on par with being back at school, not wearing teeth, or having all your pants fall out.

7

u/ZunoJ 15h ago

If it is your program, it is not your coworker but your employee. Also what about CICD, how can anybody merge code that doesn't pass tests, let alone build? Just roll back to the last commit before their changes and it's done. Takes less than a minute

1

u/frikilinux2 15h ago

Bad seniors who tell the junior to rubber stamp their code. Also not everyone uses CI/CD unfortunately

2

u/nafe42 9h ago

This is clearly the fault of DevSecFinOps

2

u/UShouldntSayThat 45m ago

That not.... but there...

Why is your CI allowing syntax errors in? Those builds should fail and deny merging

2

u/UShouldntSayThat 45m ago

and who's approving them?

1

u/another_random_bit 8h ago

CI gates exist buddy

1

u/r2k-in-the-vortex 4h ago

Thats what CI is for.

1

u/Treechcer 1h ago

How does this even happen? Why would anyone push code with syntax erros... or get it through review?

1

u/edwardbnd_99 50m ago

Larp post

1

u/slgray16 14h ago

Missing a step in your flowchart after fix all bugs.

Try adding:

Start-Sleep -Seconds 28800