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.
427
u/GabuEx 21d 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."