3 ms·
One way to do that is to open a second PR which targets the branch of the first PR. I think that's also probably the most natural way -- would you agree? I don'
by da39a3ee 3y ago
One way to do that is to open a second PR which targets the branch of the first PR. I think that's also probably the most natural way -- would you agree? I don't see it done much, but I've done it once or twice and I'd embrace it if it were more culturally standard.
- XorNot 3y agoThat's what the parent comment is talking about though: and I agree it's better, but it just pops as another global PR. The UI doesn't know what to do with it, and most developers don't really understand it without spending a bunch of time explaining it. When it works, it works great but the overhead is ridiculous since you get nothing inline on the UI: ideally something like the commits should pop up as comments, with accept/reject/discuss options.
- da39a3ee 3y agoRight, agree with all that.