Merge then review

agilob@programming.dev to Programmer Humor@programming.dev – 721 points –

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

144

You are viewing a single comment

I just commit directly to master with auto-deploy like a real cowboy, yee-haw!

Why review at all when the users will do this for you? Merge, deploy and move on. If it's broken they'll tell you.

I'm definitely going to start doing this at work. We don't want our embedded firmware for medical devices to get stale.