Hacker News new | past | comments | ask | show | jobs | submit login

I even find the "Could you" condescending.

[I think] variable definitions should be aligned so they're easier to read.




This is extremely passive. Cool you think that, but do you want them to do anything about it?

Maybe "please align the variable definitions."


It's fine for a review to be passive. The review is about the code, not the person who wrote it.

When the reviewer prescribes a solution, it requires the author to push back if they want to solve the issue differently. For example, if they noticed they could delete the variable rather than align it.

Though, I'm used to code reviews where the author and reviewer are both professionals and equals. I suppose if the author is an unknown person submitting a patch, you might want to rely less on their professional judgement.


Or just setup a linter and then you no longer have to babysit whitespace placement




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: