Hacker News new | ask | show | jobs
by aaronblohowiak 5455 days ago
We pull the branch and have the reviewer run the relevant tests. Doing this was simpler than figuring out how to do fancy stuff with the ci server (which checks the integration branch after merge.).

Having this step requires the reviewer to know which tests are relevant, which ensures that they were written or updated.