D939: remotenames: add functionality to store remotenames under .hg/hgremotenames/
durin42 (Augie Fackler)
phabricator at mercurial-scm.org
Mon Dec 4 21:15:43 EST 2017
durin42 added a comment.
In https://phab.mercurial-scm.org/D939#26379, @dlax wrote:
> pulkit (Pulkit Goyal) a écrit :
> > Updated the series with new differentials https://phab.mercurial-scm.org/D1547
> > https://phab.mercurial-scm.org/D1547 to https://phab.mercurial-scm.org/D1551
> > https://phab.mercurial-scm.org/D1551.
> Why creating new differentials? I thought the very point of Phabricator
> was to make it possible to review several versions of a given patch on
> the same entity. By abandoning and creating new differentials, we loose
> previous review feedback and it breaks the subscription group (which,
> given how handy it is subscribe to a series of differentials, is not
> really nice).
Indeed, but something happened with this series such taht I can no longer figure out what's going on. Pulkit created the new series at my request so I can review something coherent. It's a mess, but it'll unblock the review. :(
To: pulkit, #hg-reviewers, dlax, ryanmce
Cc: durin42, ryanmce, dlax, mercurial-devel
More information about the Mercurial-devel