Nature of the beast, I'm afraid. The "beast" is group programming.
First, there ought to be a firmwide linter and style-checker applied to all
changesets. This avoids a whole class of reasons to quibble.
Second, and this is more controversial, outlaw any "this is how I would have done it"
remarks, and institute a policy whereby quibblers make their proposed edits
themselves.
I need to use git as an example since that's all I know. Quibblers fetch
your PR [following this](
https://stackoverflow.com/questions/27567846/how-can-i-check...),
make their changes, and push on another branch, and ask for your counter-review.
In the vein of "talk is cheap, show me the code," verbal nitting is too easy
and hurts morale. Quibblers, if they feel strongly enough about their nits,
need to make a proper effort to "correct" your code themselves.
First, there ought to be a firmwide linter and style-checker applied to all changesets. This avoids a whole class of reasons to quibble.
Second, and this is more controversial, outlaw any "this is how I would have done it" remarks, and institute a policy whereby quibblers make their proposed edits themselves.
I need to use git as an example since that's all I know. Quibblers fetch your PR [following this]( https://stackoverflow.com/questions/27567846/how-can-i-check...), make their changes, and push on another branch, and ask for your counter-review.
In the vein of "talk is cheap, show me the code," verbal nitting is too easy and hurts morale. Quibblers, if they feel strongly enough about their nits, need to make a proper effort to "correct" your code themselves.