• m_f@midwest.social
    link
    fedilink
    arrow-up
    1
    ·
    1 year ago

    That’s 👏 what 👏 CI 👏 is 👏 for

    Warn in dev, enforce stuff like this in CI and block PRs that don’t pass. Go is just being silly here, which is not surprising given that Rob Pike said

    Syntax highlighting is juvenile. When I was a child, I was taught arithmetic using colored rods. I grew up and today I use monochromatic numerals.

    The Go developers need to get over themselves.

    • WhyEssEff [she/her]@hexbear.net
      link
      fedilink
      English
      arrow-up
      2
      ·
      1 year ago

      reading my code after being up for 18 hours and having my eyes glaze over trying to parse the structure of my monochromatic code but then I remember Rob Pike said syntax highlighting is juvenile so I throw my head against that wall for another 3 hours

      • WhyEssEff [she/her]@hexbear.net
        link
        fedilink
        English
        arrow-up
        2
        ·
        edit-2
        1 year ago

        Prescription glasses are juvenile. When I was a child, I was prescribed visual aid to help my nearsightedness. I grew up and today I raw-dog the road.

    • FlumPHP@programming.dev
      link
      fedilink
      arrow-up
      0
      ·
      1 year ago

      What’s a situation where you need an unused variable? I’m onboard with go and goland being a bit aggressive with this type of thing, but I can’t think of the case where I need to be able to commit an unused variable.

      • m_f@midwest.social
        link
        fedilink
        arrow-up
        1
        ·
        1 year ago

        You probably wouldn’t be committing this, unless you’re backing up a heavily WIP branch. The issue is that if you’re developing locally and need to make a temporary change, you might comment something out, which then requires commenting another now-unused variable, which then requires commenting out yet another variable, and so on. Go isn’t helping you here, it’s wasting your time for no good reason. Just emit a warning and allow CI to be configured to reject warnings.