this post was submitted on 14 Nov 2023
728 points (97.0% liked)

Programmer Humor

24828 readers
895 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 2 years ago
MODERATORS
728
Merge then review (programming.dev)
submitted 2 years ago* (last edited 2 years ago) by [email protected] to c/[email protected]
 

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
[–] [email protected] 20 points 2 years ago (9 children)

Before everyone loses their minds, in Extreme Programming there are safeguards other than PR reviews. Before you submit a PR, you are supposed to have written the tests and to have written your code with pair programming, so your code already has some safety measures in place. On top of that, when you merge and deploy, more tests are run, and only if all of them are green do your changes go into production.

[–] [email protected] 5 points 2 years ago (3 children)

you are supposed to have written the tests and to have written your code with pair programming,

I commented out the tests because they were failing, pipelines were green so I merged. Now it's running on prod. What do you do?

[–] [email protected] 16 points 2 years ago

Fire you for destroying the tests. It's intentional sabotage.

[–] [email protected] 6 points 2 years ago

Give you public kudos for moving fast and breaking things. We need more fearless cowboys like you around here

[–] [email protected] 1 points 2 years ago

I would fire you for incompetence and sabotage. Problem solved.

load more comments (5 replies)