Reviewing PRs is not just about the code or the tools, the way you phrase your comments matters a too, especially if you are a peer reviewer.
Should this be extracted out?rather than
Extract this outor
This should be extracted out.
This separation of X from Y is good(or
I like how you separated X from Y(or
Not sure I like)
Any reason why you did it this way instead of that way?
Please use better variable namesas
Please for the love of God can you use better variable names?. Instead, you can say
Perhaps a more intuitive variable name here?which doesn't run any risk of misinterpretation.
... I noticed the same issue in several other places too.