this post was submitted on 16 May 2025
225 points (97.9% liked)
Programmer Humor
23590 readers
1144 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
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
I had a new dev that was tasked with changing the colors of a button. took them 3 days to do it.
when I finally got the PR there were over 300 files changed and over 100k changes.
they "fixed" the while space. I asked them who asked them to do that. "Nobody, it's just maintenance on the codebase."
I then had an hour long call with them reviewing everywhere they "fixed" the whitespace and got through 10% of it. I asked them if they wanted to have another call tomorrow to review the rest or if they wanted to put the changes forward they were tasked with.
they got the hint and fixed their shit.
Does the work item describe this technical cleanup? No? PR rejected for including out of scope work.
this, this is what I am dealing with, thank you for teaching him
I get it. they're just trying to show their usefulness and skills.
I did have a talk with them after work at the bar and told them not to worry about it. the best way to show your usefulness and dedication is to do the job you were assigned to do.
there are plenty of opportunities in your career that you can use to showcase your abilities without you needing to invent them on a whim. if anything doing that will do the opposite of what you're intending.
bright kid, just needed to be hit a couple times to see how the real world works.
please tell me you have an accepted code style with proper tooling and precommit hooks set up