this post was submitted on 14 Nov 2023

730 points (97.0% liked)

Programmer Humor

18426 readers

1112 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 1 year ago

MODERATORS

Feyter@programming.dev

Vacant@programming.dev

anzo@programming.dev

BurningTurtle@programming.dev

730

Merge then review (programming.dev)

submitted 8 months ago* (last edited 8 months ago) by agilob@programming.dev to c/programmer_humor@programming.dev

146 comments fedilink hide all child comments

Move fast and break things.
Merge vulnerabilities.
Double the work.
Merge code without tests.
Anything, but don't let code become stale.

you are viewing a single comment's thread
view the rest of the comments

[–] Deifyed@lemmy.ml 1 points 8 months ago (1 children)

Sorry about the confusion. It's not sarcasm. I'm just sick and tired of people blocking my PR because of an argument about wether the function should be called X or Y or Z or D

[–] jjjalljs@ttrpg.network 2 points 8 months ago

Ah. Yeah those kind of nitpicks are annoying. We try to specify when comments are blocking or non blocking on reviews.

But I definitely block a lot of reviews over no tests, bad tests, no error handling, failed linting. And the occasional "this doesn't do what the ticket asked for"