• Scrubbles@poptalk.scrubbles.tech
    link
    fedilink
    English
    arrow-up
    27
    ·
    1 year ago

    I just had a Jr dev do this to me. Came in, so bold, and said our patterns were stupid. Told him no, while there are things to improve it’s not worth the effort to rewrite hundreds of classes over a personal preference.

    Of course when he inevitably brought it up again, I told him okay, schedule with our architect, that’s how things work after all for major rewrites. Man, was that meeting entertaining.

    Kids, we know it could be better. But we also know things like level of effort and value. Your CS330 patterns are really cool, we know them too. But we aren’t going to build them.

      • Scrubbles@poptalk.scrubbles.tech
        link
        fedilink
        English
        arrow-up
        21
        ·
        1 year ago

        Oh god this was a trigger. Had one guy just casually demand that we change our middleware layers out because it would be “more performant”. Our middleware that was rock solid, never had any issues, and no one ever noticed. He brought up how it would shave so much time, like maybe a ms or 2 when I read through what he proposed. Sure, let’s just throw out what’s been working for years without issue for a new, untested version that will take weeks to dev and test for a payout of 2 ms per request.