this post was submitted on 24 Sep 2024
838 points (98.7% liked)

Programmer Humor

19310 readers
1590 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
 
you are viewing a single comment's thread
view the rest of the comments
[–] Psaldorn@lemmy.world 33 points 1 week ago (2 children)

Merging failing tests so everybody else has failing tests and wastes time figuring out why.

Nothing neutral here

[–] jaybone@lemmy.world 9 points 1 week ago (1 children)

I haven’t played DnD in like 20 years. Is “Chaotic Dickhead” an alignment now?

[–] RisingSwell@lemmy.dbzer0.com 6 points 1 week ago

That's basically just chaotic neutral

[–] RamblingPanda@lemmynsfw.com 1 points 1 week ago (1 children)

That's what the pipeline is for. It's not that hard to pinpoint the commit that lead to the errors.

[–] Psaldorn@lemmy.world 8 points 1 week ago (1 children)

If I rebase my branch with main I do not expect any failing tests. If you waste my time merging shit code, fuck you. Fix your shit.

Unless prod is on fire and the CEO is prowling (even then, I'd argue standards should be maintained)

[–] RamblingPanda@lemmynsfw.com 5 points 1 week ago

I don't say this is good practice, you shouldn't even be able to merge to main with failing tests. I've implemented an emergency flag to do this, but I don't want to use it in normal, daily business.