Senior Dev: removes line of code. Suddenly works.
this post was submitted on 17 Sep 2025
96 points (95.3% liked)
Programmer Humor
26603 readers
3401 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
- Keep content in english
- No advertisements
- Posts must be related to programming or programmer topics
founded 2 years ago
MODERATORS
Ultimate dev: Removes 2,000 lines of code, works an order of magnitude faster.
Lol yeah.
https://youtu.be/b2F-DItXtZs?t=103
Reminds me of this but with databases. Just pipe to dev/null
We have that situation at work.
Stupid dev creates mega-PRs that do a ton of things that are not part of the ticket and thus aren't tested by QA.
When it then inevitably fails during release testing or in production, better devs need to spend quite a bit of time to figure out what the hell the stupid dev messed up and the fix is then often just a few lines.
What are the reviewers of these PRs doing? Out-of-scope PRs can be rejected and closed, no?
It's a really weird team mechanic.
We have two teams working on the same codebase. The teams are very far apart in terms of organization. We are in different departments.
That dude is in the other team. Each team does their own code review and is supposed to ask for cross-team review for stuff that affects the other team.
The guy built himself a reputation as an infallible master programmer in his team by bullying everyone else during PRs. You know, nitpicking tiny non-mistakes like variable names, blocking PRs for weeks by always finding some unnecessary detail, then waiting for a few days after that detail was adjusted to find another one and so on. So his team thinks he's the greatest. Or maybe they are afraid to be his next victim or something. Whatever the reason, they just greenlight everything he does. And he never asks for a cross-team review.
His boss also completely covers for him, so so far our escalations just went nowhere.
Can you create a paper trail about an impending issue caused by his creation, then let it fail in prod and hopefully get the right people pissed at him? Or is that more likely to backfire on you instead?
We are doing that post hoc. We have enough issues thanks to him that actually do go to production and now we are escalating again. Lets see what happens.
Good luck!
Oof sorry, that sucks.
Senior dev fix: creates a new test case alongside the fix
Hey man. There are writing coders (try things out) and thinking coders (plan things in the head, then write). None is better than the other, both have their own advantages and disadvantages.