How could one Dev commit to prod without other Devs reviewing the MR? IF you’re not protecting your prod branch that’s a cultural issue. I don’t know where you’ve worked in the past, or where you’re working now, but once it’s N+1 engineers in a code base there needs to be code reviews.
Comment on Let's blame the dev who pressed "Deploy"
Geyser@lemmy.world 3 months agoWas there a process in place to prevent the deployment that caused this?
No: blame the higher up
Yes: blame the dev that didn’t follow process
Of course there are other intricacies, like if they did follow a process and perform testing, and this still occurred, but in general…
aodhsishaj@lemmy.world 3 months ago
0x0@programming.dev 3 months ago
Oh you sweet summer child…
erwan@lemmy.ml 3 months ago
I hate to break it to you, but companies with actual safe rails to deploying to production do exist.
And when things go wrong, it’s never the responsibility on a single dev. It’s also the dev who reviewed the PR. It’s also the dev who buddy approved the deploy. It’s the whole department that didn’t have enough coverage in CI.
aodhsishaj@lemmy.world 3 months ago
I would hate to work where you developed the idea a protected main/prod branch is something novel.
j4k3@lemmy.world 3 months ago
If they didn’t follow a procedure, it is still a culture/management issue that should follow the distribution of wealth 1:1 in the company.