• limer@lemmy.dbzer0.com
    link
    fedilink
    English
    arrow-up
    13
    ·
    2 days ago

    “Sometimes the best way to fix a bug is to introduce an unstable new feature that will later have many bug reports. But the code will now work. And was only written after email chain that har management involved.”

    “This is a temporary fix only, and the feature flag it’s under should be turned off after pull request 203. Under no circumstances should bug reports 1923 and 2045 use this new feature to fix issues, even if hours of work can be saved using this ”

    “I am blameless for any future issues caused by using this new feature. Here be dragons.”

    • baines@lemmy.cafe
      link
      fedilink
      English
      arrow-up
      2
      ·
      2 days ago

      “this is temporary test code that should be removed before delivery to the customer”

      this is real

    • marcos@lemmy.world
      link
      fedilink
      arrow-up
      5
      ·
      2 days ago

      You should put this at the code, or at the flag documentation. The one place you it can’t go at all is in a commit message.

      • limer@lemmy.dbzer0.com
        link
        fedilink
        English
        arrow-up
        4
        ·
        2 days ago

        Have better docs in those places; but for a class A mess, like above, make sure the approvers see this front and center. Make them sign for it