• JackbyDev@programming.dev
    link
    fedilink
    English
    arrow-up
    16
    ·
    edit-2
    18 hours ago

    One of my colleagues submitted a PR with a bunch of emojis in the readmes and log statements and I’m just so infuriated with it.

      • JackbyDev@programming.dev
        link
        fedilink
        English
        arrow-up
        3
        ·
        13 hours ago

        There are plenty of non ASCII characters that are okay in code. ñ comes to mind. There are also box drawing characters.

        • pftbest@sh.itjust.works
          link
          fedilink
          arrow-up
          3
          ·
          11 hours ago

          We don’t use them in my project, I only added an exception for ©®™ and such. You can easily whitelist any character range you need. My command looks like this:

          - (! grep -r -I -P '[^\x{00}-\x{7f}©®™°]' src)
          
          • JackbyDev@programming.dev
            link
            fedilink
            English
            arrow-up
            3
            ·
            6 hours ago

            This is goofy, I’m not gonna fail a build because somebody used some random Unicode character. That’s draconian.

        • RagingRobot@lemmy.world
          link
          fedilink
          arrow-up
          1
          ·
          58 minutes ago

          They add excessive logging too. I had cursor write some UI code and I thought my console was going to explode. Console logs each step of the way haha. I guess that’s the easiest way for it to debug its own code