Pull requests vs. Paired Programming
2022-04-15
You NEED To Stop Using Pull Requests
TLDR:
PRs were invented for distributed projects with untrusted contributors. They are slow and inefficient and make CI/CD impossible (feature branches live for more than 1 day, which is his threshold). If you try to make it faster it becomes really annoying (because people are constantly interrupted and/or have to wait for others).
But code has to be reviewed for quality reasons. Solution: Instead of doing a separate review step after the code has been written, do the review while it is being written — i.e. Paired Programming.