this post was submitted on 09 Jul 2023
66 points (95.8% liked)

Programming

17406 readers
175 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities [email protected]



founded 1 year ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
[โ€“] Blackthorn 4 points 1 year ago (1 children)

Oh man, I hate code reviews with the strength of a thousand suns. In the enterprise world, there are so many constraints and conventions that reading other people code is even more boring than it should be. It's even worse when you LGTM it and your colleagues "accuse" you of not having read it with attention because "I can't believe you don't have any comment on my code".

[โ€“] [email protected] 2 points 1 year ago* (last edited 1 year ago)

Well, if you don't even have any comments at all, neither good nor bad, then "LGTM" doesn't ever really translate to "I have read all your code and it's wonderfule/okay enough to pass"...

To me, having worked as a developer for some years now, it means "don't got no time to look into your codes, so here's a stamp of approval to send you along your way, knowing too well that you expected me to look into your code but I didn't"

My point being, if you want to say you read all the code and it's fine, maybe refrain from using "LGTM", which is the universal sign of malicious compliance/not caring.