My thoughts on this approach:
- Yes, automate as much as possible. Lint, code formatters, and other analysis tools you may have at your disposable. The team needs to agree on the bar that must be met to submit the PR.
- Check the tests. If you've changed something, show me the tests that test that change. Most of my code review is analyzing the quality and coverage of your tests.
- I really don't care much about your actual code - the automated tools already enforced our coding standards. I'm not much for coding standards; I'll use whatever standard the team wants. I'm just glad they can be automatically enforced so I don't have to engage in pointless arguments that only serve to lower team morale.
At the end of the day, I really only care about your tests.
