| ▲ | i_cannot_hack 10 hours ago | |
There's only a limited amount of context and decisions that can be effectively communicated informally without looking at the code. Sometimes it is required that people look at the actual suggested implementation, and when doing so they might spot fundamental issues that had not been found beforehand. The conventional format for doing such a review is a PR. | ||
| ▲ | johnmaguire 9 hours ago | parent [-] | |
And thus, you don't have to build on every single unreviewed PR - but often it's quite sane to do so. | ||