this post was submitted on 16 Mar 2024
93 points (83.9% liked)

Programmer Humor

19572 readers
1995 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
 
all 12 comments
sorted by: hot top controversial new old
[–] [email protected] 34 points 8 months ago

You can stare at that code all afternoon but you won't spot the bug until a microsecond after you hit "commit".

[–] [email protected] 14 points 8 months ago (1 children)

I'm not sure I understand, what's wrong with this commit?

[–] [email protected] 10 points 8 months ago

1 change, forgot to enable the setting previously

[–] [email protected] 13 points 8 months ago (1 children)
[–] [email protected] 8 points 8 months ago* (last edited 8 months ago) (3 children)

~~Whoever committed this change thinks updating the text on the UI for this system will automagically fix issue.~~

~~It’s like editing the HTML on your bank’s website to show you have $1B in the bank 😂~~

Issue here committer forgot to update UI to expose the feature person was working on

[–] [email protected] 12 points 8 months ago (1 children)

No, the settings already existed, they just forgot to expose it

[–] [email protected] 3 points 8 months ago (1 children)

That makes sense now. Thanks for extra context.

[–] [email protected] 4 points 8 months ago

(Here's some more context: The button is in the UI, but it exposed a completely different function than advertised, probably due to being generated from copy-and-pasting the previous button)

[–] [email protected] 5 points 8 months ago (1 children)

Yeah without more context it just seemed they were renaming the setting as part of a larger diff

[–] [email protected] 1 points 8 months ago

It says "1 changed file..." at the top

[–] [email protected] 4 points 8 months ago

I'm not convinced that's it, but I don't know enough to explain why.