So my manager today asked me if I could stay later when there’s broken things in prod, and then today his star dream employee yolo’ed a full stack change into prod without review. It’s fucking massive and implements new API endpoints, touches >20 files. Many of the diffs are too large to render in the browser.

It’s almost comical, but something immediately broke.

Most of my day, I’m digging through code to identify bugs created from this shit, just to get a stealth merge midday.

I kind of don’t know what to do.

  • mesa@piefed.socialEnglish
    3·
    5 days ago

    Make ci/cd part of the process. It stops a lot of people who push huge unmaintanable changes.

    If it doesent pass the tests and cant build then it stops them from merging.

      • mesa@piefed.socialEnglish
        2·
        5 days ago

        It helps! Ive seen it with two jobs completely change the culture around code review.

        • Instead of you/team lead being the bad guy, its now the code/process.
        • Code becomes more stable and releases actually become more frequent.
        • If something goes wrong, your VM/docker/box/etc… can just be re-spun up by the same process.
        • Easier to onboard since the same build process is in the CI. And is constantly being used rather than relying on the README (that may not have been updated in a while).

        Mind you its not perfect of course. You can still “Vibe” test and/or remove tests that dont work. And make the project more brittle. Or go overboard with lint rules (I actually had to break up a fight with that one). But its much better than the old process of merge and pray.