4 ms·
> I review mostly for architectural compliance and "bad idioms" or code smells. There is difference between not like I would write it and badly written mess and
by menor 9y ago
> I review mostly for architectural compliance and "bad idioms" or code smells. There is difference between not like I would write it and badly written mess and many programmers confuse them.
This ^ I just had this discussion last week in a PR, if the comment is a matter of opinion, the whole team should agree and put a rule in the linter or style guide. If not every one is free to have their own preferences.