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.

[–] agilob@programming.dev 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?

[–] silencioso@lemmy.world 16 points 2 years ago

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

[–] pomodoro_longbreak@sh.itjust.works 6 points 2 years ago

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

[–] Blackthorn@programming.dev 1 points 2 years ago

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

load more comments (5 replies)