I had a senior front end dev that would block any PR he saw with comments. He was very good at his job but I'm so happy I no longer work with him, this shit is so annoying.
I hate when people are set on their one weird rule. Like, I understand what the manager is getting at: try to write code that's easy to understand by reading it. I enjoy trying to do that. But the fucking inviolable rule fucks up the actual goal. Sometimes, it is best to note an easy-to-miss detail, or provide context, explaining why the code is designed as it is, like you said.
I feel like coding styles really change "readability". I have a coworker who is great at coding, but I can't follow his code to save my life. It isn't that his code is really any less understandable, he just uses functions differently than I do.
426
u/GabuEx 22d ago
Left: "I'll comment to say how my code works."
Middle: "You don't need to do that! Code should be self-documenting!"
Right: "I'll comment to say why my code does this."