- ddh ( @DarkDarkHouse@lemmy.sdf.org ) English133•11 months ago
As your future colleague wondering what the hell that variable is for, thanks Go.
- Willem ( @Willem@kutsuya.dev ) 46•11 months ago
I prefer for it to be just a warning so I can debug without trouble, the build system will just prevent me from completing the pull request with it (and any other warning).
- CodeBlooded ( @CodeBlooded@programming.dev ) 63•11 months ago
If this language feature is annoying to you, you are the problem. You 👏are 👏 the 👏 reason 👏 it 👏 exists.
I worked in places where the developers loaded their code full of unused variables and dead code. It costs a lot of time reasoning about it during pull request and it costs a lot of time arguing with coworkers who swear that they’re going to need that code in there next week (they never need that code).
This is a very attractive feature for a programming language in my opinion.
PS: I’m still denying your pull request if you try to comment the code instead.
❗️EDIT: A lot of y’all have never been to programming hell and it shows. 🪖 I’m telling you, I’ve fixed bayonets in the trenches of dynamically typed Python, I’ve braved the rice paddies of CICD YAML mines, I’ve queried alongside SQL Team Six; I’ve seen things in production, things you’ll probably never see… things you should never see. It’s easy to be against an opinionated compiler having such a feature, but when you watch a prod deployment blow up on a Friday afternoon without an easy option to rollback AND hours later you find the bug after you were stalled by dead code, it changes you. Then… then you start to appreciate opinionated features like this one. 🫡
- m_f ( @m_f@midwest.social ) 50•11 months 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.
- FlumPHP ( @flumph@programming.dev ) 4•11 months 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 ( @m_f@midwest.social ) 13•11 months 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.
- hare_ware ( @hare_ware@pawb.social ) 7•11 months ago
I will need it two minutes tops. If I don’t use it by then, I’ll delete it, especially if it gives a warning like Rust does. But this? It just gets in the way.
- jormaig ( @jormaig@programming.dev ) 7•11 months ago
Have you looked at the post? Use case: you are testing something or playing around and you want to try something. That’s supper common
- spookedbyroaches ( @spookedbyroaches@lemm.ee ) 2•11 months ago
I agree that golang is being dumb when you don’t even have the option to tell it that this is a testing env or something. But the thing about syntax highlighting is not the same. One is about handholding the developer so much that it makes it even more difficult to develop, and the other is a completely optional feature that is so uselful and non intrusive that even wizardly editors like emacs use it.
- Urik ( @Urik@lemmy.ca ) 28•11 months ago
That’s a problem with your workplace, not the language nor OP.
You could have a build setting for personal development where unused variables are not checked, and then a build setting for your CI system that will look for them. It gives you freedom to develop the way you want without being annoyed when you remove something just to test something, but will not merge your PR unless the stricter rules are met.- LittleLordLimerick ( @LittleLordLimerick@lemm.ee ) 4•11 months ago
Why in the world would you want to develop something that doesn’t follow the coding rules required by your org, just so you can go back and fix everything before submitting a PR? That’s just extra work.
- CodeBlooded ( @CodeBlooded@programming.dev ) 3•11 months ago
I concur, it is a problem with that workplace. (In this case, OP is just sharing a funny meme. I wouldn’t suggest this meme means they’re a problem. I could have made this meme and I love the feature.)
Developing on a team at a company is like the “Wild West.” What’s considered to be acceptable will not only vary from workplace to workplace, but it can also fluctuate as developers and managers come and Go. Each of them have their own unique personality with their own outlook on what “quality” code looks like. (And many of them do not care about code quality whatsoever. They just need to survive 1-2 years there, make management happy with speedy deliveries, and then they can move on to the next company with a 30% pay bump.)
Having experienced working with developers who frequently filled with code base with unused code while having no control over who will leave or join as a contributor to the code base, I think features like this make for a more sane development experience when you’re developing with a team of seemingly random people that you never personally invited to contribute to the code base.
will not merge your PR unless the stricter rules are met.
This doesn’t fly when you work in big corporate and the boss doesn’t care about the code meeting stricter rules. “A working prototype? No it’s not- that’s an MVP! Deploy it to production now and move onto the next project!”
- thanks_shakey_snake ( @thanks_shakey_snake@lemmy.ca ) 16•11 months ago
Lol new copypasta unlocked 🔓
- CodeBlooded ( @CodeBlooded@programming.dev ) 4•11 months ago
🫡
- redcalcium ( @redcalcium@lemmy.institute ) 11•11 months ago
It costs a lot of time reasoning about it during pull request and it costs a lot of time arguing with coworkers who swear that they’re going to need that code in there next week (they never need that code).
You should go to your team leader and ask them to enforce a coding standard. I agree with other commenters that said this should be a warning instead of an error.
- ricojo777 ( @ricojo777@lemmy.sdf.org ) 3•11 months ago
I’ll start calling SQL “squeal” now in the spirit of this comment
- Magnetar ( @Magnetar@feddit.de ) 4•11 months ago
No amount of propaganda will ever get me to pronounce it “sequel”. I’ll die in that hill.
- CodeBlooded ( @CodeBlooded@programming.dev ) 2•11 months ago
Wait… how do you say it? es queue el?
- Magnetar ( @Magnetar@feddit.de ) 7•11 months ago
I do, unashamedly.
- Nato Boram ( @NatoBoram@lemm.ee ) English22•11 months ago
And I fucking love it. Thank you Go!
- stebo ( @stebo02@lemmy.dbzer0.com ) 24•11 months ago
lol what’s wrong with you
- Nato Boram ( @NatoBoram@lemm.ee ) English3•11 months ago
“Other people” are what’s wrong with me. People don’t use linters/formatters/type annotations when it’s optional and produce dogshite code as a result. Having the compiler itself enforce some level of human decency is a godsend.
- stebo ( @stebo02@lemmy.dbzer0.com ) 1•11 months ago
sure but unused variables are no biggie
- lightsecond ( @lightsecond@programming.dev ) 19•11 months ago
You go Go!
- yum13241 ( @yum13241@lemm.ee ) 17•11 months ago
This makes me not want to use Golang at all.
- jeanma ( @jeanma@lemmy.ninja ) English14•11 months ago
OP never said he/she commits such code but wants to iterate, test, explore.
Of course, unused var should not be part of a commit.
- MonkderZweite ( @MonkderZweite@feddit.ch ) 9•11 months ago
Comment the unused variable out and no security hole gets accidentally shipped.
- LBRABO ( @LBRABO@lemmy.sdf.org ) 4•11 months ago
¯\_(ツ)_/¯
- nothendev ( @nothendev@sopuli.xyz ) 1•10 months ago
Zig