Displaying 2 results from an estimated 2 matches for "d48017".
Did you mean:
148017
2020 Jan 15
3
[cfe-dev] Phabricator -> GitHub PRs?
Hi David,
On Wed, Jan 15, 2020 at 12:51 PM David Greene via llvm-dev
<llvm-dev at lists.llvm.org> wrote:
> I would like to understand better how people use Phab's advanced
> features and why. For example, what drives the need for patch series
> and dependence relationships? Some walk-through examples would be very
> helpful.
Here's a somewhat more complex example of
2020 Jan 16
2
[cfe-dev] Phabricator -> GitHub PRs?
...d reasons as line noise in
later commits) that an earlier comment is not relevant.
>
> > Conversely, having the linkage between different commits provides
> > context to reviewers.
>
> I can see that too. For commits that are dependent on a long series of
> other commits (D48017 for example), how hard is it to review without
> reviewers also looking at the commits upon which it depends? Do people
> really review such commits early in the process? Even without the early
> review I agree the context can be useful.
>
> > It is also helpful to me: I can kee...