[skywhopper]

Nah, automated testing cover basic functionality. For most PRs, a senior familiar with the code wouldn’t need to check it out and manually test anything, that’s not what “code review” is most of the time. If you need them to look at the code in a running state, that should be part of the CI process, not a manual task for the developer.

A good reviewer can call out bad strategic coding decisions or misinterpretations of the requirements. QA is another layer of review entirely.