And they might very well be! But just make sure you get a rapport with the reviewed party, and prefix or suffix the comment with a "just a suggestion" or an explicit "I'm not asking you to act on this remark". With a semantic review, you can shorten it to "nit:".
That's what the article suggests, using "Suggestion:" as a prefix.
> or an explicit "I'm not asking you to act on this remark"
Same, the article suggests the "Remark:" prefix. Both are more terse than what you suggest here.
* I've heard Kiwis say "F*ck you?" in lieu of "Seriously?"
* A Brit who says "That is an interesting solution!" usually does not feel intellectually stimulated by the solution but is conveying that it is utter garbage.
Thus, I very much like the proposed idea of well-defined labels.*
If in work setting your coworkers (god forbid teamleads or managers) interact like that just run, don't try to solve it with conventional reviews or something. Believe me you will not finish blinking before they start misusing this too to cope with their repressed anger through mockery/shallow sarcasm. "nitpick: have you considered escaping user input?"
I can be sarcastic but I have never directed sarcasm towards someone who I am reviewing. (Sarcasm towards third-party code is okay for me.)
I don't know if people who are using sarcasm would want to start labeling things like that as such, since it tends to detract from the intended effect.
Disclaimer: not a Brit.
The label makes it easy to find comments still in need of replies, and for automated security to prevent merging items with outstanding crucial comments.
The label opens the possibility of automated reporting and/or review of reviews.
I also enjoy a tactical use of smileys and other emojis where applicable :)