| ▲ | newshackr 4 hours ago | |||||||
What if you want someone to look at a portion of it but they don't know enough to approve the whole thing. They give +1 Someone else knows the other portion well and sees the +1 and decides to +2. In practice this ends the stalemate where partial owners don't feel confident to approve the whole thing | ||||||||
| ▲ | chii 4 hours ago | parent [-] | |||||||
The PR needs to have someone who knows the whole thing. Having several people review each separate parts but not understanding the others' can cause interaction bugs. If such bugs cannot happen (say, due to modularity, or type safety guarantees etc), then it won't be the case where you need to have a partial approve. I am not a fan of partial approve. Either you think the code is approvable, or it isn't. | ||||||||
| ||||||||