alt Hacker NewsRight, 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".
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".
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.