• Dave.
    link
    fedilink
    349 months ago

    You can stare at that code all afternoon but you won’t spot the bug until a microsecond after you hit “commit”.

    • @malloc
      link
      English
      8
      edit-2
      9 months ago

      Whoever committed this change thinks updating the text on the UI for this system will automagically fix issue.

      It’s like editing the HTML on your bank’s website to show you have $1B in the bank 😂

      Issue here committer forgot to update UI to expose the feature person was working on

      • AatubeOP
        link
        fedilink
        129 months ago

        No, the settings already existed, they just forgot to expose it

        • @malloc
          link
          English
          39 months ago

          That makes sense now. Thanks for extra context.

          • AatubeOP
            link
            fedilink
            49 months ago

            (Here’s some more context: The button is in the UI, but it exposed a completely different function than advertised, probably due to being generated from copy-and-pasting the previous button)

      • @[email protected]
        link
        fedilink
        59 months ago

        Yeah without more context it just seemed they were renaming the setting as part of a larger diff

        • AatubeOP
          link
          fedilink
          19 months ago

          It says “1 changed file…” at the top

      • Victor
        link
        49 months ago

        I’m not convinced that’s it, but I don’t know enough to explain why.