2 ms·
Is this just a way of saying, “stop wasting all your review time on the style guide and look at the system design”? Although, the style guide should just be fo
by barbariangrunge 3y ago
Is this just a way of saying, “stop wasting all your review time on the style guide and look at the system design”?
Although, the style guide should just be followed and fixed before you get to code review phase. That’s just a matter of professionalism
- int0x2e 3y agoAt a former team, we went from spending quite a bit of time on code style comments and disagreements to spending no time at all on it, with the simple act of making the code linter a breaking step in our CI build, and deciding no review will start until the build is green. We had to adjust our linter settings here and there - but it was still super efficient for everyone's time compared to what we had before... I can't recommend this more.