D2048: utils: copy util.py as utils/dateutil.py

durin42 (Augie Fackler) phabricator at mercurial-scm.org
Thu Feb 8 09:15:14 EST 2018


durin42 added a comment.


  In https://phab.mercurial-scm.org/D2048#34978, @lothiraldan wrote:
  
  > In https://phab.mercurial-scm.org/D2048#34743, @durin42 wrote:
  >
  > > Can we do all of https://phab.mercurial-scm.org/D2048::https://phab.mercurial-scm.org/D2056 as a single change? What you've done means we have no useful blame information on everything done in https://phab.mercurial-scm.org/D2049::D0256...
  >
  >
  > Moving function one by one proved very useful in the past when refactoring like this triggered subtle bugs in core or extension. Having each move on its changeset make it easier to bisect the part that breaks other and save a great amount of time when that happens.
  
  
  Life's tough. I care more about the blame history on this code.
  
  (Other reviewers are welcome to overrule me, but I don't find this argument even slightly convincing.)

REPOSITORY
  rHG Mercurial

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

To: lothiraldan, #hg-reviewers, durin42
Cc: durin42, mercurial-devel


More information about the Mercurial-devel mailing list