• RustyNova@lemmy.world
    link
    fedilink
    arrow-up
    1
    ·
    10 days ago

    Well it’s not that simple… Because whoever wrote that made it way too complicated (and the production version has been tweaked without updating the dev too)

    A clean rewrite with some guard clauses helped remove the haduken ifs and actually zipping the file outside of the zipped directory helped a lot

    • r00ty@kbin.life
      link
      fedilink
      arrow-up
      0
      ·
      10 days ago

      I mean, I have to say I’ve hastened my own demise (in program terms) by over-engineering something that should be simple. Sometimes adding protective guardrails actually causes errors when something changes.

      • Croquette@sh.itjust.works
        link
        fedilink
        arrow-up
        1
        ·
        7 days ago

        Which are what guardrails are for. When something change, you don’t know the impact the change will have.

        By having guardrails, you make sure to limit/eliminate potential critical issues.