Good article, definitely agree with the points here other than separate PR for tests. I always like to keep the tests with the code they are for. I have taken to separate PRs for model, ui and integration though. That works well to keep PRd small