search for: d48017

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...