I run every PR as part of my code review
I run every PR as part of my code review. Why? Because often, the PR doesn't even work right.
Often a PR only works on a "happy path". It ignores validation, error handling, accessibility, loading state, etc.
It's best to catch these issues *before* merge.
"That's what QA is for".
Well many dev teams don't have dedicated QA. But even if you do have QA, throwing incomplete work over the wall to QA is an anti-pattern.
The dev team should write code as if QA doesn't exist.
Assuring quality is everyone's job.
"We should trust each other".
People make mistakes. Even the most senior devs. So we shouldn't blindly trust each other.
Everyone's work should be reviewed before merge.