Hey all! My team at work is struggling with growing pains of getting into a formalized review process, so I was wondering if any of you guys have some things to live or die by in your code reviews. How much of it is manual, or how much is just static code analysis + style guide stuff, etc?


deleted by creator
How many developers do you have that you have to disable merging? Or is it more a safety-net?
deleted by creator
Merging straight from feature -> master is gutsy
deleted by creator