• RustyNova@lemmy.world
    link
    fedilink
    arrow-up
    24
    ·
    edit-2
    9 months ago

    Not a go dev. Is it really preventing compilation or is it just some hardened linting rules? Most languages can prevent compile on those errors if tweaked, but that seems bad if it’s not a warning

    • xmunk@sh.itjust.works
      link
      fedilink
      arrow-up
      60
      ·
      9 months ago

      Yes, and it fucking sucks. It’s a great thing to lint for but it makes debugging such a pain - commenting out an irrelevant block to focus your debugging will sometimes break your ability to compile… it’s extremely jarring.

      • technojamin@lemmy.world
        link
        fedilink
        arrow-up
        14
        ·
        9 months ago

        This is why many languages have errors and warnings as separate things. Errors for things that for sure prevent the program from working, and warnings for things that are probably wrong but don’t prevent things from working. If you have a setting to then treat warnings as errors (like for CI checks), then you get all the guarantees and none of the frustration.

      • herrvogel@lemmy.world
        link
        fedilink
        arrow-up
        8
        ·
        9 months ago

        Have they given an explanation as to why that is? I mean why make it a fatal error that prevents compilation, when you could make it a warning and have the compiler simply skip it?

        • YIj54yALOJxEsY20eU@lemm.eeOP
          link
          fedilink
          arrow-up
          8
          ·
          edit-2
          9 months ago

          Its an effort to keep large code bases clean. I think they should allow them when running go run but not when building.

          • RustyNova@lemmy.world
            link
            fedilink
            arrow-up
            5
            ·
            9 months ago

            I can see the sentiment here… Going through 100 clippy warning on Rust is just not fun… I know there’s the good old clippy --fix but I’m paranoid it breaks my code accidentally.

            Could probably have a compromise like 5 unused variables and your code don’t compile

            • Faresh@lemmy.ml
              link
              fedilink
              English
              arrow-up
              2
              ·
              9 months ago

              but I’m paranoid it breaks my code accidentally

              Automated tests and version control should prevent that from being a problem, I imagine.

          • Ethan@programming.dev
            link
            fedilink
            English
            arrow-up
            3
            ·
            edit-2
            9 months ago

            I totally agree that it’s really annoying when debugging, but go run literally builds then executes. I think what they should do is add a build flag. So debug builds can pass that flag to get the builder to shut up, and leave it those errors enabled for production builds.

          • expr@programming.dev
            link
            fedilink
            arrow-up
            2
            arrow-down
            1
            ·
            9 months ago

            Or, you know, treat it as a warning like literally every other language. There’s absolutely no good reason for it to prevent a build outright, but then again, there’s not really good reasons for many of the decisions behind go.

        • frezik@midwest.social
          link
          fedilink
          arrow-up
          5
          ·
          9 months ago

          Keep in mind that this is the same language that prefers function names ToBeLikeThis(), and the reason is that it looks different than Java.

          • fadhl3y@lemmy.world
            link
            fedilink
            English
            arrow-up
            4
            ·
            9 months ago

            Every time I think “perhaps I should give Golang another try”, it’s shit like this that keeps me noping out

            • YIj54yALOJxEsY20eU@lemm.eeOP
              link
              fedilink
              arrow-up
              2
              ·
              9 months ago

              There’s two types of programming languages, the ones people complain about and the ones nobody uses. Go is still my most productive language and is killer for building webservers. I basically use it as a scripting language since it’s so fast to write, compile, and execute.

      • Valmond@lemmy.mindoki.com
        link
        fedilink
        arrow-up
        4
        ·
        9 months ago

        Whoah, that seems like you’d flesh out code elsewhere, you know when you throw stuff together to make it work, and then fix it up to standards.

        Feels like you should have to make git commits perfectly well before being able to compile…

        Put that overwhelmingly intrusive thing in a hook checking out your commits instead (when you push your branch ofc).

        • Ethan@programming.dev
          link
          fedilink
          English
          arrow-up
          1
          ·
          9 months ago

          You get used to it. The only time I really notice it these days is when I’m debugging and commenting out code.

            • Ethan@programming.dev
              link
              fedilink
              English
              arrow-up
              1
              ·
              9 months ago

              *when I’m doing debugging that requires commenting out code.

              Most of the time, I don’t comment out code. I run the code in a debugger, step through it, and see how the behavior deviates from what I expect. I mostly only resort to commenting out code if I’m having trouble figuring out where the problem is coming from, which isn’t that often.

    • YIj54yALOJxEsY20eU@lemm.eeOP
      link
      fedilink
      arrow-up
      6
      ·
      9 months ago

      I don’t think its inherently bad but it feels jarring when the language allows you reference nill pointers. It’s so effective in its hand holding otherwise that blowing things up should not be so easy.