[dshacker]

Right, I think there is always a balance between being strict on code reviews, and just letting people ship stuff. I've also seen the other end of the stick in which a senior employee is blocking an important pr over "spacing".

[AndrewDucker]

Your software linting should be automated, and if possible it should be formatted automatically.

It really shouldn't be possible to have arguments in a PR over formatting.