D939: remotenames: add functionality to store remotenames under .hg/hgremotenames/

durin42 (Augie Fackler) phabricator at mercurial-scm.org
Tue Dec 5 02:15:43 UTC 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. :(

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D939

To: pulkit, #hg-reviewers, dlax, ryanmce
Cc: durin42, ryanmce, dlax, mercurial-devel


More information about the Mercurial-devel mailing list