"Stop Nitpicking in Code Reviews" by Dan Lew https://blog.danlew.net/2021/02/23/stop-nitpicking-in-code-reviews/
Totally agree with this. I've tried to stop nitpicking when it's just something like "you could do this in 1 line of code instead of 3." If it doesn't really matter, why fill up the PR with noise?
@nolan One thing I wish GitHub would change is to rename the "Approve" and "Request changes" options for a review. There's a lot of things (nitpicks, but also slightly more serious issues) that can very easily be resolved, and that don't need another re-review because you can trust your collaborator to deal with them on their own using their best judgment.
"Request changes" only really applies when you need to align on the approach before merging it in, which is very rare.
@nolan "Trusting your collaborator to deal with feedback on their own using their best judgement" also means that it's OK for them to brush it aside.
That attitude, combined with suggested edits making it really easy for them to incorporate your feedback if they see the value, does away with most of the friction nitpicks and other minor disagreements can cause.
Server run by the main developers of the project It is not focused on any particular niche interest - everyone is welcome as long as you follow our code of conduct!